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

✨ Integrating release workflow - initial build, test, package, and upload artifacts to the ci runs. #34

Merged
merged 44 commits into from
Oct 3, 2024

Conversation

savitharaghunathan
Copy link
Member

@savitharaghunathan savitharaghunathan commented Sep 30, 2024

This workflow builds the package, runs tests, and packages the extension. It also uploads it to the GH ci runs. It runs on merge and can be manually triggered on branches main and release-*. Supports linux, macos, windows.
ref issue: #11
todo:

  • Add Konveyor-analyzer binaries
  • Auto Publish it to the marketplace

Sample run - https://github.com/savitharaghunathan/editor-extensions/actions/runs/11111525940

I had to update the Readme as the vsce package fails if we use template generated readme file.

Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
…tion

Revert ":sparkles: adding initial release workflow"
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
…ad artifacts to the ci runs.

Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
@savitharaghunathan
Copy link
Member Author

Ref issue: #11

Copy link
Member

@djzager djzager left a comment

Choose a reason for hiding this comment

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

This looks great. Thank you for making quick work of this. Only minor changes requested.

If you could please create an issue with all that you know to be required to publish to the VSCode marketplace and/or the Open VSX Registry, that would be fantastic.

.github/workflows/release.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Signed-off-by: Savitha Raghunathan <6026527+savitharaghunathan@users.noreply.github.com>
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
savitharaghunathan and others added 2 commits October 1, 2024 15:13
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Signed-off-by: Savitha Raghunathan <6026527+savitharaghunathan@users.noreply.github.com>
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Copy link
Member

@djzager djzager left a comment

Choose a reason for hiding this comment

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

I really like moving this into ci-repo. A few comments.

.github/workflows/ci-repo.yml Outdated Show resolved Hide resolved
.github/workflows/ci-repo.yml Outdated Show resolved Hide resolved
.github/workflows/ci-repo.yml Outdated Show resolved Hide resolved
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Copy link
Member

@djzager djzager left a comment

Choose a reason for hiding this comment

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

I think this is the last set of changes I'll need to ask for. Thank you for your patience, I really had not expected that we could make this so consumable for other stakeholders this quickly...really great work.

My requested changes reflect what I think to be the best option. You should also update the nightly workflow like:

jobs:
  nightly:
    uses: ./.github/workflows/ci-repo.yml
    with:
      shouldRelease: true

That will give us v0.0.1-dev+20241003 releases as early as tomorrow night that can with downloadable vscode extensions our stakeholders can install and use. Exceeds all of my expectations.

.github/workflows/ci-repo.yml Outdated Show resolved Hide resolved
.github/workflows/ci-repo.yml Outdated Show resolved Hide resolved
.github/workflows/ci-repo.yml Outdated Show resolved Hide resolved
.github/workflows/ci-repo.yml Outdated Show resolved Hide resolved
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
djzager
djzager previously approved these changes Oct 3, 2024
Copy link
Member

@djzager djzager left a comment

Choose a reason for hiding this comment

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

Nice! A few formatting comments that may be impactful.

.github/workflows/ci-repo.yml Outdated Show resolved Hide resolved
.github/workflows/ci-repo.yml Outdated Show resolved Hide resolved
formatting fixes
dropping release name

Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
@savitharaghunathan
Copy link
Member Author

@djzager ptal when you get some time. I am not able to test the release step on my fork. do you have any thoughts? Also, what do you mean by this -

Nice! A few formatting comments that may be impactful.

comments for steps?

Copy link
Member

@djzager djzager left a comment

Choose a reason for hiding this comment

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

LGTM

@savitharaghunathan savitharaghunathan merged commit 3984bdc into konveyor:main Oct 3, 2024
13 checks passed
@savitharaghunathan savitharaghunathan deleted the release_auto branch October 8, 2024 11:51
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