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

docs: update script for issue dashboard #775

Merged
merged 1 commit into from
Jul 22, 2024

Conversation

sdiazlor
Copy link
Contributor

@sdiazlor sdiazlor commented Jul 9, 2024

Screenshot 2024-07-09 at 13 57 14
Screenshot 2024-07-09 at 13 57 17
Screenshot 2024-07-09 at 13 57 19

@sdiazlor sdiazlor linked an issue Jul 9, 2024 that may be closed by this pull request
Copy link

codspeed-hq bot commented Jul 9, 2024

CodSpeed Performance Report

Merging #775 will not alter performance

Comparing docs/768-docs-update-issue-dashboard (4461578) with main (fe615d6)

Summary

✅ 1 untouched benchmarks

@davidberenstein1957
Copy link
Member

@sdiazlor, LFTM.

Do you think it might perhaps make sense to create an installable GH repo to hold this logic, and avoid needing to update it in argilla and distilabel whenever there might be minor changes like this? And I guess it is also a nice general creative workflow on how to manage community issues in docs that you would be able to share on socials if you feel comfortable )

Copy link
Member

@davidberenstein1957 davidberenstein1957 left a comment

Choose a reason for hiding this comment

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

see comment

@sdiazlor
Copy link
Contributor Author

@davidberenstein1957 Yes, I believe that's a great idea. By the moment, I'd merge this one and update later with the proper new logic.

@sdiazlor
Copy link
Contributor Author

Hi, @davidberenstein1957. By the moment, could you approve so we can merge?

@davidberenstein1957 davidberenstein1957 self-requested a review July 22, 2024 09:55
@davidberenstein1957
Copy link
Member

@sdiazlor I missed this.

@davidberenstein1957 davidberenstein1957 merged commit b5b9e60 into main Jul 22, 2024
7 checks passed
@davidberenstein1957 davidberenstein1957 deleted the docs/768-docs-update-issue-dashboard branch July 22, 2024 09:55
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.

[DOCS] Update issue dashboard
2 participants