Skip to content

Now both workflows read the full version directly from the manifest:#10

Merged
bigbansal merged 1 commit into
mainfrom
codex-ide-feature-foundation
May 28, 2026
Merged

Now both workflows read the full version directly from the manifest:#10
bigbansal merged 1 commit into
mainfrom
codex-ide-feature-foundation

Conversation

@bigbansal
Copy link
Copy Markdown
Contributor

  • VSCode: apps/vscode/package.json version
  • JetBrains: apps/jetbrains/gradle.properties pluginVersion

Before publishing, each workflow checks that the vscode-v<version> / jetbrains-v<version> tag does not already exist locally or on origin, and fails fast with a clear message asking the maintainer to bump the manifest if it does. Gradle no longer needs -PpluginVersion= overrides (it reads gradle.properties), and the VSCode workflow no longer rewrites package.json in the workspace.

Summary

Linked Issue

Closes #

Testing

  • pnpm lint
  • pnpm build
  • pnpm test

Checklist

  • The PR is focused on one issue or cohesive change.
  • New behaviour has tests where applicable.
  • Documentation is updated where needed.
  • DCO sign-off is included in every commit.
  • Screenshots or screen recordings are attached for UI changes.
  • Security-sensitive changes avoid logging secrets or source code.

- VSCode: apps/vscode/package.json `version`
- JetBrains: apps/jetbrains/gradle.properties `pluginVersion`

Before publishing, each workflow checks that the `vscode-v<version>` /
`jetbrains-v<version>` tag does not already exist locally or on origin,
and fails fast with a clear message asking the maintainer to bump the
manifest if it does. Gradle no longer needs `-PpluginVersion=` overrides
(it reads gradle.properties), and the VSCode workflow no longer rewrites
package.json in the workspace.
@bigbansal bigbansal merged commit 5e860e1 into main May 28, 2026
3 checks passed
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.

1 participant