Skip to content

fix: remove mojo-toml from the failed compat data file#247

Merged
carolinefrasca merged 1 commit intomodular:mainfrom
sstadick:fix/failed_compat_file
Apr 1, 2026
Merged

fix: remove mojo-toml from the failed compat data file#247
carolinefrasca merged 1 commit intomodular:mainfrom
sstadick:fix/failed_compat_file

Conversation

@sstadick
Copy link
Copy Markdown
Collaborator

@sstadick sstadick commented Apr 1, 2026

Everything after and including 1186270 is failing to build packages (and therefore to push them) because of this failure.

https://github.com/modular/modular-community/actions/runs/23751157619/job/69193322212

Successfully built recipe mojo-toml
Removed mojo-toml from failed-compatibility.json
Successfully built recipe mojo-regex
...
Run command: git push --set-upstream origin main
Command failed
stderr: remote: error: GH013: Repository rule violations found for refs/heads/main.        
remote: Review all repository rules at https://github.com/modular/modular-community/rules?ref=refs%2Fheads%2Fmain        
remote: 
remote: - Changes must be made through a pull request.        
remote: 
To https://github.com/modular/modular-community
 ! [remote rejected] main -> main (push declined due to repository rule violations)
error: failed to push some refs to 'https://github.com/modular/modular-community'

Note that this is still just the same issue that's been hit repeatedly, which is that the action can't push to main, so this whole failed-compat branch of code does this every time.

@sstadick sstadick self-assigned this Apr 1, 2026
@sstadick sstadick requested a review from carolinefrasca April 1, 2026 09:50
@sstadick
Copy link
Copy Markdown
Collaborator Author

sstadick commented Apr 1, 2026

All packages that have failed should be pushed when this gets merged in.

Copy link
Copy Markdown
Member

@carolinefrasca carolinefrasca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@carolinefrasca carolinefrasca merged commit f52f2dd into modular:main Apr 1, 2026
3 of 4 checks passed
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.

2 participants