Skip to content

Conversation

@jacoscaz
Copy link
Contributor

@jacoscaz jacoscaz commented May 2, 2025

This PR tweaks the grabrepos.py script to default to the version of the specification tagged with the most-recently-created tag when a user clicks on the 'specification' tab.

I propose this change as I believe it would help with clarity and fostering the ecosystem around Homie. Once V5 will have been published, it would make more sense for first-time users to land on the current version of the spec rather than the development version as implementations should target the former.

Furthermore, this made for a good excuse to learn how to build the website locally.

EDIT: I hadn't noticed there was an issue for this! Closes #27 .

…sion of the spec when a user clicks on the 'specification' tab
@Tieske Tieske requested a review from Copilot May 3, 2025 06:53
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the grabrepos.py script to default to the most recently created specification tag rather than the develop branch.

  • The diff sorts repo tags by commit date in descending order instead of lexicographically.
  • It changes the default file copied for the specification landing page to use the newest tag rather than "develop".

@Tieske Tieske requested a review from schaze May 3, 2025 06:54
Copy link
Contributor

@Tieske Tieske left a comment

Choose a reason for hiding this comment

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

@schaze I just gave you privileges. This lgtm, please check and merge if ok.

Copy link
Contributor

@schaze schaze left a comment

Choose a reason for hiding this comment

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

Looks good to me too. I am just checking the code "manually" - I do not have a local environment setup so I trust @jacoscaz tested this adequately.

@schaze schaze merged commit e8c57d8 into homieiot:master May 3, 2025
1 check passed
@schaze
Copy link
Contributor

schaze commented May 3, 2025

@jacoscaz : It works! Thanks a lot for this. I now get v4 spec when navigating to the homieiot page and select specification.

@jacoscaz
Copy link
Contributor Author

jacoscaz commented May 4, 2025

@schaze @Tieske thank you both! Having these simple PRs reviewed and merged is very encouraging for getting further invested in Homie.

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.

Default version shown is "develop"

3 participants