Windows Related Changes

[lizgzil]

@Aiden-RC 's changes from https://github.com/nestauk/ojd_daps_skills/pull/183 to allow the package to run on Windows

---

Thanks for contributing to Nesta's Skills Extractor Library 🙏!

If you have suggested changes to code anywhere outside of the ExtractSkills class, please consult the checklist below.

Checklist ✔️🐍:

• [ ] I have refactored my code out from notebooks/
• [ ] I have checked the code runs
• [ ] I have tested the code
• [ ] I have run pre-commit and addressed any issues not automatically fixed
• [ ] I have merged any new changes from dev
• [ ] I have documented the code
  • [ ] Major functions have docstrings
  • [ ] Appropriate information has been added to READMEs
• [ ] I have explained the feature in this PR or (better) in output/reports/
• [ ] I have requested a code review

If you have suggested changes to documentation (and/or the ExtractSkills class), please ALSO consult the checklist below.

Documentation Checklist ✔️📚:

• [ ] I have run make html in docs
• [ ] I have manually reviewed the docs/build/*.html files locally to ensure they have formatted correctly
• [ ] I have pushed both relevant files AND their corresponding docs/build/*.html files

[Aiden-RC]

You could change the action to only run on pull request, and make any changes to the dev branch required to be pushed through a pull request which will fix the double trigger of the unit test as sadly you can't do push or pull

[lizgzil]

Hi @Aiden-RC - after a discussion with the team we decided to remove the checks triggering on a pull request, and added some info about this to the readme. Thanks so much for helping with this!!

[Aiden-RC]

Hi @Aiden-RC - after a discussion with the team we decided to remove the checks triggering on a pull request, and added some info about this to the readme. Thanks so much for helping with this!!

Yeah thats a better way of doing it than double triggering actions and it's all good :)