-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
Shift the GitHub action to CLI repository #396
Comments
bounty/candidate |
@Shurtu-gal I wish to work on this issue. Basically, want to try something new, apart from website 😅 |
Great, however would need to wait for it to be accepted before assigning. |
Bounty Issue's service commentText labels: @asyncapi/bounty_team The Bounty Program is not a Mentorship Program. The accepted level of Bounty Program Participants is Middle/Senior.Third-party contributors should coherently articulate how they are going to approach the resolution process when expressing a desire to work on this Bounty Issue. |
@asyncapi/bounty_team as per the rules with maintainers having more preference, I have assigned @akshatnema the issue. |
Thanks @Shurtu-gal 😄 👍 |
Bounty Issue's Timeline
Please note that the dates given represent deadlines, not specific dates, so if the goal is reached sooner, it's better.Keep in mind the responsibility for violations of the Timeline. |
Hey @aeworxet, |
Upon request of the Bounty Program Participant (@akshatnema), all remaining target dates of the Bounty Issue's Timeline are extended by two calendar weeks. Bounty Issue's Timeline Extended
Please note that the dates given represent deadlines, not specific dates, so if the goal is reached sooner, it's better.Keep in mind the responsibility for violations of the Timeline. |
Thanks @aeworxet |
@akshatnema do you need any help on this ? |
@akshatnema, please provide an update to the PR. |
1 similar comment
@akshatnema, please provide an update to the PR. |
@akshatnema had scheduled a call with @derberg on 2024-08-27 to clarify some final details on this Bounty Issue before merging asyncapi/cli#1487. |
Response, critical for technical resolution of this Bounty Issue (clarification on implementation) was delayed for one period of three consecutive working days so all remaining target dates of the Bounty Issue's Timeline are extended by two calendar weeks. Bounty Issue's Timeline Extended
Please note that the dates given represent deadlines, not specific dates, so if the goal is reached sooner, it's better.Keep in mind the responsibility for violations of the Timeline. |
Hey @aeworxet , |
Upon request of the Bounty Program Participant (@akshatnema), all remaining target dates of the Bounty Issue's Timeline are extended by two calendar weeks. Bounty Issue's Timeline Extended
Please note that the dates given represent deadlines, not specific dates, so if the goal is reached sooner, it's better.Keep in mind the responsibility for violations of the Timeline. |
@akshatnema, please provide an update to the PR. |
|
The latest conversation with @akshatnema took place on Friday 2024-09-20 in Slack, where he stated that the solution to this Bounty Issue will be merged later because it requires thorough testing, as it is a complete repo being moved into an existing one, and then became unresponsive. Reasons for sudden silence after a lively conversation can be numerous, so I'm waiting for the clarification on this matter before taking any action. |
Hey @aeworxet Sorry for the late reply. The solution requires thorough testing, as this is a complete repo being moved into an existing one, so I need an extension by 2 weeks. But most probably, I will do everything on this weekend. |
Upon request of the Bounty Program Participant (@akshatnema), all remaining target dates of the Bounty Issue's Timeline are extended by two calendar weeks. Bounty Issue's Timeline Extended
Please note that the dates given represent deadlines, not specific dates, so if the goal is reached sooner, it's better.Keep in mind the responsibility for violations of the Timeline. |
@aeworxet the versions automation task will be done in another PR. |
@aeworxet this issue is pretty complicated as some things related to containerizing part is still not working and we still have to do the automation of version locking and bumping. @akshatnema is also pretty new to the CLI repo so is it possible to reclassify this bounty as P.S: Original Source: asyncapi/cli#1487 (comment) |
Upon request of the AsyncAPI Maintainer (@Shurtu-gal, Bounty Issue was Reclassified and its Timeline Extended
Please note that the dates given represent deadlines, not specific dates, so if the goal is reached sooner, it's better.Keep in mind the responsibility for violations of the Timeline. |
Implementation of #396 (comment)
is the PR |
Timeline for this Bounty Issue has run out. Knowing it is quite complex, and taking into account that both @akshatnema and @Shurtu-gal have remained responsive during its resolution, I must first ask what should be done with it moving forward? Is more time needed to complete it, is it in the final stages, or is there something else? |
It's in the final stages, just need to get the automation in place. |
This bounty is still in progress. We have been discussing possible solutions and have come with two plausible ones:-
cc: @aeworxet |
@aeworxet @akshatnema is exploring the above mentioned solutions and would be giving any updates he finds out here. |
This Bounty Issue is still in progress and is encountering many restrictions and limitations related to GitHub Actions, AsyncAPI infrastructure, and the release process. |
I tried to create release workflow using changeset here - https://github.com/akshatnema/cli/actions/runs/11999463258/job/33447396874. Currently debugging in it to create a changeset in the fork repo, and test the workflow. |
Why do we need this improvement?
Currently the versions, releases and other stuff are isolated from CLI which is directly coupled with this CLI. However, this creates problems in bumping of versions and keeping code updated as well.
Furthermore, any change in CLI spec could be easily made here as well.
How will this change help?
This will help in automation of releases of CLI and tie them together properly.
How could it be implemented/designed?
This issue can be resolved through the following steps:-
.github
directory.github-action-for-cli/entrypoint.sh
Line 29 in 590180c
github-action-for-cli/Dockerfile
Line 13 in 590180c
action.yml
in root folder.Additional steps:
Important
The action and bump of version would need to be tested in your personal repo or account. From my past experience would suggest using an alternate account itself.
🚧 Breaking changes
Yes
The text was updated successfully, but these errors were encountered: