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.

Sign up for GitHub

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

Lint only changed files on PR. #494

Closed
2 tasks done
fredrikhgrelland opened this issue Aug 24, 2021 · 0 comments · Fixed by #495
Closed
2 tasks done

Lint only changed files on PR. #494

fredrikhgrelland opened this issue Aug 24, 2021 · 0 comments · Fixed by #495
Labels
change/patch Only used on PRs. Automatic releases with github actions if tagged. theme/ci Related to build, test, and release pipeline type/enhancement New feature or request
Milestone
0.11.1

Comments

@fredrikhgrelland
Copy link
Contributor

fredrikhgrelland commented Aug 24, 2021 •
edited
Loading

Feature description

Run linter only on the files affected by the PR

Why is it needed?

Less time more clarity in PR.

Suggestion(s)/solution(s) [Optional]

Lint full codebase on schedule.
Lint only changes on PR.

Definition of done

^

Checklist (after created issue)

  • Added label(s)
  • Added to milestone
@fredrikhgrelland fredrikhgrelland added type/enhancement New feature or request change/patch Only used on PRs. Automatic releases with github actions if tagged. theme/ci Related to build, test, and release pipeline labels Aug 24, 2021
@fredrikhgrelland fredrikhgrelland added this to the 0.11.1 milestone Aug 24, 2021
fredrikhgrelland pushed a commit that referenced this issue Aug 24, 2021
Lint only changed files on Pull Requests #494
4a0800b
@fredrikhgrelland fredrikhgrelland mentioned this issue Aug 24, 2021
Merged
5 tasks
fredrikhgrelland pushed a commit that referenced this issue Aug 24, 2021
Lint only changed files on Pull Requests #494
4e76c23
fredrikhgrelland pushed a commit that referenced this issue Aug 24, 2021
Lint only changed files on Pull Requests #494
c4a5521
fredrikhgrelland added a commit that referenced this issue Aug 24, 2021
@fredrikhgrelland
Merge pull request #495 from Skatteetaten/linter_change
5d58314 
Lint only changed files on Pull Requests #494
fredrikhgrelland pushed a commit that referenced this issue Aug 24, 2021
Lint only changed files on Pull Requests #494
aba26eb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
change/patch Only used on PRs. Automatic releases with github actions if tagged. theme/ci Related to build, test, and release pipeline type/enhancement New feature or request
Projects
None yet
Milestone
0.11.1
Development

Successfully merging a pull request may close this issue.

Lint only changed files on Pull Requests #494 Skatteetaten/vagrant-hashistack
1 participant
@fredrikhgrelland