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

Jump to bottom

Check manifest: pre-commit, fixes, cleaning #1683

Merged
merged 2 commits into from May 30, 2022
Merged

Check manifest: pre-commit, fixes, cleaning #1683

merged 2 commits into from May 30, 2022

Conversation

jensens
Copy link
Contributor

@jensens jensens commented May 30, 2022 •
edited

  • (re)move dup .md files
  • ad pre-commit hook for check-manifest
  • fix Manifest itself

@jensens jensens requested a review from ericof May 30, 2022 19:35
@codeclimate
Copy link

codeclimate bot commented May 30, 2022

Code Climate has analyzed commit 563472e and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (0% is the threshold).

This pull request will bring the total coverage in the repository to 100.0% (0.0% change).

View more on Code Climate.

@ericof ericof added the CI/CD This issue/PR relates to the project CI/CD update or fix. label May 30, 2022
@jensens jensens merged commit ba1c461 into master May 30, 2022
@jensens jensens deleted the check-manifest branch May 30, 2022 19:38
@eruber eruber mentioned this pull request Feb 5, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ericof ericof ericof approved these changes

Assignees
No one assigned
Labels
CI/CD This issue/PR relates to the project CI/CD update or fix.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@jensens @ericof