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

Add QuIC Repolinter check #89

Closed
wants to merge 1 commit into from
Closed

Add QuIC Repolinter check #89

wants to merge 1 commit into from

Conversation

quic-nwtn
Copy link

This GitHub Action runs Repoliner on push or pull requests to master. This should've been enabled when the project was first setup. We need to enable this and correct violations (ignore the Warnings and focus on the Errors). In some cases, the repolinter rules need to be tweaked depending on the project, which we can discuss.

Signed-off-by: quic-nwtn <162972850+quic-nwtn@users.noreply.github.com>
@androm3da
Copy link

@quic-nwtn I don't know that the repolinter is a good fit for this repo. It has a great deal of upstream content that should stay as-is.

If the repolinter should get applied, it will probably need some significant changes to the rules it uses.

@quic-nwtn
Copy link
Author

@mynameistechno , @androm3da , please Note that repolinter runs and checks every repositories under quic for violations. let me see if we need to tweak the rules.

@mynameistechno
Copy link

Nitish, I should've mentioned that we don't want to add this check on Forked repos. I.e. only on QuIC-authored repos. Sorry about that!

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.

3 participants