common-workflow-lab/wdl-cwl-translator

Update readme with instructions for running tests

kinow opened this issue · 7 comments

kinow commented

From @mr-c on Matrix,

tox -e py39-pydocstyle
make diff_pydocstyle_report
make install-deps

I think this can go to the README.md?

ntachukwu commented

Awesome. Thanks

ntachukwu commented 
remote: error: GH006: Protected branch update failed for refs/heads/main.
remote: error: 16 of 16 required status checks are expected.
To github.com:common-workflow-lab/wdl-cwl-translator.git
 ! [remote rejected] main -> main (protected branch hook declined)
error: failed to push some refs to 'github.com:common-workflow-lab/wdl-cwl-translator.git'

I can't seem to push to the main branch. Do I need to create a new branch first?

kinow commented

Looks like @Th3nn3ss . Can you create a branch and a pull request, please? When you create the PR, try the following: start the pull request with "Closes #108". You will notice GitHub adds some underline in "Closes".

The pull request will be linked to this issue, and once merged, this pull request will be automatically closed 👍

kinow commented

If you are on main, and you have committed changes, you can try the following:

  • git checkout -b new-branch-name (choose a good name for your branch)
  • git push -u origin new-branch-name

Then create the pull request, and fix your main branch:

  • git checkout main
  • git reset --hard origin/main

I think this might work? Or you can start from scratch on a new branch, up to you.

ntachukwu commented

This is amazing. Will do that.

ntachukwu commented

make install-deps throws an error because it should be make install-dep.

ntachukwu commented

make install-deps throws an error because it should be make install-dep.

  • 👍2