Skip to content

Conversation

@elylucasctfl
Copy link
Contributor

Summary

Description

Motivation and Context

Checklist (check all before merging)

  • Both unit and integration tests are passing
  • There are no breaking changes
  • Changes are reflected in the documentation

When adding a new method:

  • The new method is exported through the default and plain CMA client
  • All new public types are exported from ./lib/export-types.ts
  • Added a unit test for the new method
  • Added an integration test for the new method
  • The new method is added to the documentation

@elylucasctfl elylucasctfl marked this pull request as ready for review November 14, 2025 15:53
@elylucasctfl elylucasctfl requested a review from a team as a code owner November 14, 2025 15:53
Comment on lines +74 to +80
- name: Get latest release tag
id: get-tag
run: |
TAG=$(gh api repos/${{ github.repository }}/releases/latest --jq .tag_name)
echo "tag=$TAG" >> $GITHUB_OUTPUT
env:
GITHUB_TOKEN: ${{ steps.vault.outputs.GITHUB_TOKEN }}
Copy link
Contributor

Choose a reason for hiding this comment

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

I've just started noticing that this is not getting the correct tag for the summary in the next step.
https://github.com/contentful/contentful-management.js/actions/runs/19354829208#summary-55378237318

I'm putting together a PR in the OSS template that uses git describe instead to get the correct tag. https://github.com/contentful/open-source-project-template/pull/33/files

Can update now or wait til later since our other repos will need to be updated also.

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.

4 participants