Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI: GHA Test #173

Merged
merged 4 commits into from
Dec 7, 2019
Merged

CI: GHA Test #173

merged 4 commits into from
Dec 7, 2019

Conversation

dgasmith
Copy link
Collaborator

@dgasmith dgasmith commented Dec 1, 2019

Description

Tries out the GHA CI pr integration and updates the status badge.

Changelog description

Status

  • Ready to go

@dgasmith
Copy link
Collaborator Author

dgasmith commented Dec 1, 2019

Lint check isn't picking up configuration settings and failing. Very odd.

@dgasmith
Copy link
Collaborator Author

dgasmith commented Dec 2, 2019

iSort isn't picking up setup.cfg config, but black is which is the important one. Skipping isort/autoflake for now.

We originally pushed back against the lint checker as we were worried that it would hinder submissions as too complex. As we now have the make format command this seems pretty straight forward these days and we can mark it as not required for PRs that we feel need to be merged over perfect linting. Any thoughts here?

Copy link
Collaborator

@loriab loriab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm! glad to see an example for this.

@@ -7,4 +7,5 @@
<!-- Provide a brief single sentence for the changelog. -->

## Status
- [ ] `make format` run on the codebase (may have to `conda install black isort autoflake -c conda-forge` or `pip install black isort autoflake`)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I took the liberty of adding this directly to your branch, so feel free to tweak/object.

@loriab
Copy link
Collaborator

loriab commented Dec 6, 2019

oh, and there's a .yapfignore file that should probably go.

@dgasmith dgasmith merged commit c0a2b5c into MolSSI:master Dec 7, 2019
@dgasmith dgasmith deleted the ghav2 branch December 7, 2019 18:47
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