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 workflow to compute Hasura permission diff using Metadelta #19716

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

silky
Copy link

@silky silky commented Feb 29, 2024

What does this PR do?

Hello :)

I've built a (free for open-source) tool to help out with Hasura permission changes. In developing it, I found this repository as a big user of Hasura.

So, I thought I'd make a PR to add an automatic step to compute a permission diff, following the guide - https://invariant.club/guides/getting-started-with-metadelta.html.

This doesn't address any particular issue on the repo, and I won't be offended if you ignore it, but I thought I'd prepare this PR anyway, in case you would like to try it out!

Motivation

This repo uses Hasura extenstively, and in particular has frequent hasura-related permission changes that are (probably) hard to review from the source-code changes alone.

More

It'd be extremely valuable for me to get feedback on the tool I've built; and in particular any features you may want/think are missing.

Please let me know! I'm always happy to chat and can be reached via the email on my GitHub profile :)

Copy link

welcome bot commented Feb 29, 2024

Thank you for this contribution! Could you please confirm if you already sent in the signed Contributor License Agreement? See https://docs.bigbluebutton.org/support/faq.html#why-do-i-need-to-sign-a-contributor-license-agreement-to-contribute-source-code Thanks in advance!

@silky silky changed the title workflow to compute the diff Add workflow to compute Hasura permission diff using Metadelta Feb 29, 2024
@silky
Copy link
Author

silky commented Feb 29, 2024

Yes I've signed the CLA.

@antobinary
Copy link
Member

Thank you very much for this contribution! The CLA was received too!

@silky
Copy link
Author

silky commented Mar 2, 2024

I've simplified this PR now to use the new GitHub Action - https://github.com/marketplace/actions/hasura-permission-diff-with-metadelta - instead of hard-coding the workflow :)

Copy link

sonarcloud bot commented Mar 2, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@antobinary antobinary added this to the Release 3.1 milestone Mar 7, 2024
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.

None yet

2 participants