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

Use awesome-lint #148

Merged
merged 3 commits into from
Dec 1, 2023
Merged

Use awesome-lint #148

merged 3 commits into from
Dec 1, 2023

Conversation

vordimous
Copy link
Collaborator

@vordimous vordimous commented Nov 22, 2023

This PR fixes lint errors, adds a separate license file, and creates a PR job to lint addttions.

awesome-data-engineering % npx awesome-lint .
✖ Linting

  README.md:1:1
  ✖  1:1  The repository should have "awesome" as a GitHub topic       remark-lint:awesome-github
  ✖  1:1  The repository should have "awesome-list" as a GitHub topic  remark-lint:awesome-github
  ✖  1:1   License was not detected by GitHub                           remark-lint:awesome-github

  3 errors

@vordimous
Copy link
Collaborator Author

@igorbarinov I cannot add topics to this repo to satisfy the above lint rule. Once you do that, this PR can be merged.

@vordimous vordimous mentioned this pull request Nov 30, 2023
@igorbarinov igorbarinov merged commit 3d8c9dc into master Dec 1, 2023
1 check failed
@vordimous vordimous deleted the implement-awesome-lint branch December 1, 2023 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants