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

Checking lint in PR CI checks #1023

Merged
merged 4 commits into from
Nov 1, 2023
Merged

Conversation

BamButz
Copy link
Contributor

@BamButz BamButz commented Oct 23, 2023

Related issue

Closes #987

Context / Background

As described in the issue, I added linting to the CI checks to avoid introduction of linting faults.

What change is being introduced by this PR?

Checks workflow now checks for linting issues.

PR #1019 should be merged first to fix some lint errors.

@BamButz
Copy link
Contributor Author

BamButz commented Oct 23, 2023

I will rebase as soon as #1019 is merged in, that should fix the checks.

@araujoarthur0
Copy link
Collaborator

@BamButz now it should be ok

@BamButz
Copy link
Contributor Author

BamButz commented Oct 28, 2023

Changelog Bot does not comply with your linting rules: see commit 322a4d3

@araujoarthur0
Copy link
Collaborator

Yes :( saw that earlier today #1027

@araujoarthur0
Copy link
Collaborator

Think that might be a blocker otherwise every PR will restart the issue.

@araujoarthur0
Copy link
Collaborator

Can you try now again to rebase? It might have been a problem only when the file had no changes for a release - it is no longer empty.

Copy link

codecov bot commented Oct 31, 2023

Codecov Report

Merging #1023 (7c9f09c) into main (ae757ab) will not change coverage.
Report is 1 commits behind head on main.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1023   +/-   ##
=======================================
  Coverage   75.11%   75.11%           
=======================================
  Files          26       26           
  Lines        2202     2202           
  Branches      346      346           
=======================================
  Hits         1654     1654           
  Misses        548      548           

@araujoarthur0 araujoarthur0 merged commit 3f7076e into thamara:main Nov 1, 2023
8 checks passed
@araujoarthur0
Copy link
Collaborator

\changelog-update

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.

Checking lint in PR CI checks
2 participants