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

chore: Add revanced-bot to the exclusion list #233

Merged
merged 1 commit into from
Apr 18, 2024
Merged

Conversation

validcube
Copy link
Member

@validcube validcube commented Mar 27, 2024

Not sure what to label this commit, It's not really considered a "feature" or a bug "fix". I'll use chore as miscellaneous.

@revanced-bot has been engaged in ReVanced Manager repository due to Translation CI but not to the point where it should appear in the contributor list as it only recent (4 days ago as of writing), so it would make sense to add @revanced-bot to the exclusion list like semantic-release-bot is.

@oSumAtrIX
Copy link
Member

Usually this kind of implementation is problematic. What if the API is changed? If different users commit then this filter has no effect. #137 fixes the issue by proposing to add a frontend configuration to the API which would allow configuring the site to match the API. For now this is fine

@Ushie Ushie merged commit f86456c into ReVanced:dev Apr 18, 2024
0 of 2 checks passed
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