Skip to content

Added a testing case that should resolve to BOG - #429

Merged
comcon1 merged 4 commits into
mainfrom
autocomplete-metadata-test
Jun 21, 2026
Merged

Added a testing case that should resolve to BOG#429
comcon1 merged 4 commits into
mainfrom
autocomplete-metadata-test

Conversation

@mdondrup

Copy link
Copy Markdown
Collaborator

TESTING molecule metadata should be resolved to BOG and add comments automatically.

Comment thread Molecules/membrane/TESTING/metadata.yaml Outdated
mdondrup added 3 commits June 21, 2026 20:52
3
3
3
Reviewdog indeed made suggestions in each run. But it was set to fail upon any differece
due to a misunderstanding of what the fail_level meant.

Now, we set
 fail_level: none         # post suggestions without failing the job

Also made the output of the python script visible for debugging.

The changes need to be merged into main to become effective.
@mdondrup
mdondrup requested a review from comcon1 June 21, 2026 19:16
@mdondrup

mdondrup commented Jun 21, 2026

Copy link
Copy Markdown
Collaborator Author

Hi @comcon1. To test this, the wf needs to be in main, so please approve. We may need a few more rounds, so expect some more quick merge requests.

@comcon1 comcon1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let's try

@comcon1
comcon1 merged commit 0510246 into main Jun 21, 2026
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.

2 participants