Skip to content

Conversation

@sdesrozis
Copy link
Contributor

Description:

Add a Windows cmd version of the script run_code_style. I'm not an expert of such a batch language but it seems to work pretty well as far I have tested.

Check list:

  • New tests are added (if a new feature is added)
  • New doc strings: description and/or example code are in RST format
  • Documentation is updated (if required)

@vfdev-5
Copy link
Collaborator

vfdev-5 commented Nov 16, 2021

@sdesrozis thanks for the PR, can't we use bash scripts on windows as we probably be able to do on ci ?
Otherwise, should it also work with powershell ?

@sdesrozis
Copy link
Contributor Author

Yes, it works well with powershell. Using bash should be a good option, but it is not the usage in my company. Note that a win make.bat script is provided for documentation generation. I assumed that such scripts could be introduced as they really make the work easier.

Anyway, it increases the maintenance, I completely understand if this PR is rejected.

@vfdev-5
Copy link
Collaborator

vfdev-5 commented Nov 16, 2021

Ok for me to have it, thanks Sylvain !

@vfdev-5 vfdev-5 merged commit 7430897 into pytorch:master Nov 16, 2021
@sdesrozis sdesrozis deleted the run_code_style_win branch November 17, 2021 07:13
fco-dv pushed a commit to fco-dv/ignite that referenced this pull request Nov 23, 2021
Ishan-Kumar2 pushed a commit to Ishan-Kumar2/ignite that referenced this pull request Dec 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants