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

fix: correct change set #726

Merged
merged 2 commits into from
Aug 21, 2024
Merged

fix: correct change set #726

merged 2 commits into from
Aug 21, 2024

Conversation

marufrasully
Copy link
Contributor

In this PR #719 it looks like we had a comma in change set. Therefore there is a suspicion that pipe line build https://github.com/SAP/ui5-language-assistant/actions/runs/10490758568/job/29066687441 fails. This is because packages could not be incremented. With this PR which only include correct change set file, it should fix that issue.

Copy link

changeset-bot bot commented Aug 21, 2024

⚠️ No Changeset found

Latest commit: 4225f88

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes changesets to release 13 packages
Name Type
@ui5-language-assistant/vscode-ui5-language-assistant-bas-ext Patch
vscode-ui5-language-assistant Patch
@ui5-language-assistant/context Patch
@ui5-language-assistant/language-server Patch
@ui5-language-assistant/logic-utils Patch
@ui5-language-assistant/user-facing-text Patch
@ui5-language-assistant/xml-views-completion Patch
@ui5-language-assistant/xml-views-definition Patch
@ui5-language-assistant/xml-views-quick-fix Patch
@ui5-language-assistant/xml-views-tooltip Patch
@ui5-language-assistant/xml-views-validation Patch
@ui5-language-assistant/binding Patch
@ui5-language-assistant/fe Patch

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
Contributor

@vadson71 vadson71 left a comment

Choose a reason for hiding this comment

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

The incorrect changeset file saved in the repo and will disturb the next build. In normal situation all changeset files are deleted and new release version is generated.
I'd suggest to correct the existing broken file instead of creating the new one

@marufrasully marufrasully changed the title fix: add change set only fix: correct change set Aug 21, 2024
Copy link

sonarcloud bot commented Aug 21, 2024

Copy link
Contributor

@vadson71 vadson71 left a comment

Choose a reason for hiding this comment

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

Comment addressed, Let's go.

@marufrasully marufrasully merged commit ce17641 into master Aug 21, 2024
9 checks passed
@marufrasully marufrasully deleted the fix/change-set branch August 21, 2024 20:33
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.

2 participants