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

[BUG] adopters.json data file is not updated #2805

Closed
2 tasks done
sambhavgupta0705 opened this issue Mar 23, 2024 · 8 comments · Fixed by #2864
Closed
2 tasks done

[BUG] adopters.json data file is not updated #2805

sambhavgupta0705 opened this issue Mar 23, 2024 · 8 comments · Fixed by #2864
Labels

Comments

@sambhavgupta0705
Copy link
Member

sambhavgupta0705 commented Mar 23, 2024

Describe the bug.

So basically we first add data in adopters.yml file and then it gets updated in adopters.json file but the last time when the data was updated in yml file the json file wasn't updated.

Expected behavior

adopters.json should also get updated as on each and every commit by anyone the file gets updated

👀 Have you checked for similar open issues?

  • I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

Are you willing to work on this issue ?

No, someone else can work on it

@akkshitgupta
Copy link
Contributor

Hey @sambhavgupta0705, I have already worked with yaml file in previous PR. Also, the task can be achieved using a CI workflow, which gets triggered whenever the particular file is changed in a pull request. I would like to solve this particular issue :)

@sambhavgupta0705
Copy link
Member Author

Yeah we can think of CI workflow for this
What are your thoughts on this @akshatnema @anshgoyalevil @derberg

@akshatnema
Copy link
Member

Yeah, updating it via workflow is a good solution. We can add the updation steps in existing workflow of newsroom and meetings, or we can do it weekly by adding it in tools workflow, depending upon how we want to update it.

@sambhavgupta0705
Copy link
Member Author

sambhavgupta0705 commented Mar 29, 2024

Well I think the Yaml is being updated manually so I think we should update the json file weekly but I have one doubt that the adopters.json file is present in .gitignore so will it create any problem

@akkshitgupta
Copy link
Contributor

Any updates? I would like to solve the bug :)

CC: @sambhavgupta0705 @akshatnema

@akshatnema
Copy link
Member

@akkshitgupta Yeah, you can work on this issue.

@akkshitgupta
Copy link
Contributor

akkshitgupta commented Apr 14, 2024

@akshatnema @sambhavgupta0705, Since the adopters.yml file is updated with the new Adopters of AsyncAPI with a new pull request. Would it not be a good idea to trigger the workflow whenever there is a change in the file specifically rather than executing the task repeatedly every day without having even a single change in the already existing adopters.[json/yml] file.

@sambhavgupta0705
Copy link
Member Author

@akkshitgupta we don't need adopters.json file as it is a build time file and it is not required to have that in the master codebase that is being handled automatically during the build time

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants