Skip to content

Update maven.yml with checkout for badges branch#81

Merged
Ahsanadam merged 2 commits into
mainfrom
issue-76
Feb 10, 2022
Merged

Update maven.yml with checkout for badges branch#81
Ahsanadam merged 2 commits into
mainfrom
issue-76

Conversation

@FelixJacobsen

@FelixJacobsen FelixJacobsen commented Feb 8, 2022

Copy link
Copy Markdown
Contributor

Solves #76

Creates a new .svg file as a image. The images are stored in a dedicated branch called "branch". In that way the images doesn't require two review approvals for each push.

@ToniKaru ToniKaru self-requested a review February 8, 2022 14:06
@ToniKaru ToniKaru removed their request for review February 9, 2022 09:51
@ToniKaru

ToniKaru commented Feb 9, 2022

Copy link
Copy Markdown
Contributor

I can't do a proper review of the yml today so I took myself off the list of Reviewers since this is acute, but I did see the stuff that Martin showed us, as in:

  1. That the branch should be called badges and yours is.
  2. That the branch should be completely empty as yours is.
    Good luck!

@@ -16,18 +16,27 @@ jobs:

steps:
- uses: actions/checkout@v2

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It may be that line 18 is a duplication of line 20 and should be deleted.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No it's two different checkouts. First checkout on line 18 does a checkout of main branch. Then the branch badges is checked out to folder badges :)

@Ahsanadam Ahsanadam merged commit ecb9313 into main Feb 10, 2022
@Ahsanadam Ahsanadam deleted the issue-76 branch February 10, 2022 03:12
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.

5 participants