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

feat: Include verification for main branch #22

Closed
wants to merge 2 commits into from
Closed

feat: Include verification for main branch #22

wants to merge 2 commits into from

Conversation

codeis4fun
Copy link

What?

Include verification for main branches, as we are using commit version for our old gitlab and new gitlab repository.

Type of change?

  • [feat]: Include verification for main branch
  • [test]: Include unit tests for main branch

Why?

We are migrating bot projects from our old repository to the newest gitlab and its default branch is main, not master. This changes is supposed to support both projects with master and main default branches.

How?

Include verification on the commit message looking for specific strings contained in it.

How has this been tested?

  • Unit tests

Before submitting a PR please make sure to check if:

  • The code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented on my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • The changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and/or existing unit tests pass locally with my changes
  • New and/or existing integration tests pass locally with my changes

@FabioCaffarello
Copy link

Nice work!

@esequielvirtuoso
Copy link
Collaborator

@codeis4fun Great job. Thanks!

@esequielvirtuoso
Copy link
Collaborator

@codeis4fun

I'm closing this PR once I've added the same code in another PR.

It has been released on the 2.0.3 version.

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