Skip to content

Implement workflow for standardized commits.#67

Merged
SyntaxSpin merged 17 commits into
Sparkleseditor:mainfrom
Jeiel0rbit:workflow
Jun 12, 2025
Merged

Implement workflow for standardized commits.#67
SyntaxSpin merged 17 commits into
Sparkleseditor:mainfrom
Jeiel0rbit:workflow

Conversation

@Jeiel0rbit

@Jeiel0rbit Jeiel0rbit commented Jun 12, 2025

Copy link
Copy Markdown
Contributor

@SyntaxSpin Please configure at https://github.com/Sparkleseditor/Sparkles-app/settings/rules/new?target=branch to not allow merging unless all CI checks have successfully completed. If you are unsure how to configure this, I request permission to make the changes.

@SyntaxSpin

Copy link
Copy Markdown
Collaborator

it is not showing for me

@Jeiel0rbit

Copy link
Copy Markdown
Contributor Author

it is not showing for me

Wait...

@Jeiel0rbit Jeiel0rbit force-pushed the workflow branch 3 times, most recently from 4ed79fe to 481a38a Compare June 12, 2025 14:58
@Jeiel0rbit

Copy link
Copy Markdown
Contributor Author

@SyntaxSpin https://github.com/Sparkleseditor/Sparkles-app/settings/rules/new?target=branch

Please try again. Fix the code. It is ready to merge. But first configure more branch rule settings or become a collaborator.

@SyntaxSpin

Copy link
Copy Markdown
Collaborator

Oh I got it it's not Sparkles-app anymore

@SyntaxSpin

Copy link
Copy Markdown
Collaborator

@Jeiel0rbit So what's the rule give a description please

@Jeiel0rbit

Copy link
Copy Markdown
Contributor Author

Oh I got it it's not Sparkles-app anymore

No. It is to facilitate what each commit does. All contributors will be required.

@Jeiel0rbit

Copy link
Copy Markdown
Contributor Author

@Jeiel0rbit So what's the rule give a description please

I replied via Telegram. Then just merge this code.

@SyntaxSpin SyntaxSpin merged commit 5e55329 into Sparkleseditor:main Jun 12, 2025
1 check 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