Skip to content

Conversation

@tpoliaw
Copy link
Collaborator

@tpoliaw tpoliaw commented Nov 6, 2024

No description provided.

@tpoliaw tpoliaw force-pushed the ci branch 4 times, most recently from dc7271a to 127604a Compare November 7, 2024 09:20
@tpoliaw tpoliaw force-pushed the ci branch 2 times, most recently from 1ec6f38 to 1e0b8d5 Compare November 28, 2024 15:09
@tpoliaw tpoliaw force-pushed the ci branch 4 times, most recently from 0a6f834 to 5651505 Compare November 28, 2024 16:33
Copy link
Contributor

@callumforrester callumforrester left a comment

Choose a reason for hiding this comment

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

LGTM, only a few minor comments

@@ -0,0 +1,58 @@
name: Numtracker CI
Copy link
Contributor

Choose a reason for hiding this comment

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

Out of scope but can we decide if this project is called "numtracker" or "data endpoint lookup" and sort out all the references in this repo. Not opinionated on the verdict.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I don't think either name really says what it does and numtracker was shorter. Agree there should probably only be one name though.

On PRs, run
* Check formatting
* Check clippy lints
* Build everything
* Run tests
@tpoliaw tpoliaw merged commit eeadde2 into main Nov 29, 2024
3 checks passed
@tpoliaw tpoliaw deleted the ci branch November 29, 2024 14:04
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.

3 participants