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

ci: add a new workflow to validate pr title #312

Merged
merged 4 commits into from
Dec 15, 2024

Conversation

BoltonDev
Copy link

New workflow to enforce PR title following these conventions: https://www.conventionalcommits.org/en/v1.0.0.

@VALERA771
Copy link

Maybe add it in pull_request_opened workflow?

@BoltonDev
Copy link
Author

I thought about it yeah, but we need to change the name Labeler to another thing but idk.

@VALERA771
Copy link

VALERA771 commented Dec 14, 2024

There is a file named pull_request_opened. Contains labeler and assigner. So name of file is applicable for this job also

@BoltonDev
Copy link
Author

BoltonDev commented Dec 14, 2024

No, there are 2 jobs, set-labelers and assigner but the name of the workflow is Labeler
https://github.com/ExMod-Team/EXILED/blob/master/.github/workflows/pull_request_opened.yml#L1

@BoltonDev BoltonDev merged commit e930454 into ExMod-Team:master Dec 15, 2024
5 checks passed
BoltonDev added a commit that referenced this pull request Dec 15, 2024
* ci: add a new workflow to validate pr title

* refactor: edit `pull_request_opened` workflow to add "valide PR title" check

* refactor: remove previous workflow for pr title

* refactor: a small change on the name of each job
louis1706 added a commit that referenced this pull request Dec 27, 2024
* Delete .github/dependabot.yml

* Update push_nuget.yml

* Update dev.yml

* ci: add a new workflow to validate pr title (#312)

* ci: add a new workflow to validate pr title

* refactor: edit `pull_request_opened` workflow to add "valide PR title" check

* refactor: remove previous workflow for pr title

* refactor: a small change on the name of each job

* refactor: missing branches target

* XMAS NW_documentation

* Update Chamber.cs

* Update Exiled.API.csproj

* Update EXILED/Exiled.API/Exiled.API.csproj

Co-authored-by: Yamato <66829532+louis1706@users.noreply.github.com>

* Update Chamber.cs

* Update Locker.cs

* Update Locker.cs

* Oops

* Update Locker.cs

* Update Locker.cs

---------

Co-authored-by: Misaka-ZeroTwo <45165615+Misaka-ZeroTwo@users.noreply.github.com>
Co-authored-by: Bolton <48883340+BoltonDev@users.noreply.github.com>
Co-authored-by: Yamato <louismonneyron5@yahoo.com>
Co-authored-by: Yamato <66829532+louis1706@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants