Skip to content

Conversation

@lucaswerkmeister
Copy link
Member

No description provided.

@lucaswerkmeister
Copy link
Member Author

IIUC, to actually do the release we just push a tag and it ends up on Packagist automatically?

@codecov-commenter
Copy link

codecov-commenter commented Oct 21, 2022

Codecov Report

Merging #76 (0ee3012) into master (24be7c8) will not change coverage.
The diff coverage is n/a.

❗ Current head 0ee3012 differs from pull request most recent head bffa98e. Consider uploading reports for the commit bffa98e to get more accurate results

@@            Coverage Diff            @@
##             master      #76   +/-   ##
=========================================
  Coverage     88.40%   88.40%           
  Complexity       72       72           
=========================================
  Files             6        6           
  Lines           138      138           
=========================================
  Hits            122      122           
  Misses           16       16           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Member

@mariushoch mariushoch left a comment

Choose a reason for hiding this comment

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

Looks good and complete otherwise.

@JeroenDeDauw JeroenDeDauw merged commit 1084142 into master Oct 21, 2022
@JeroenDeDauw JeroenDeDauw deleted the release-3.1.0 branch October 21, 2022 13:58
@JeroenDeDauw
Copy link
Member

Tagged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants