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

Chore/do not use old endpoints #254

Merged
merged 3 commits into from
Oct 25, 2024
Merged

Chore/do not use old endpoints #254

merged 3 commits into from
Oct 25, 2024

Conversation

snim2
Copy link
Contributor

@snim2 snim2 commented Oct 25, 2024

Update docs and test to use current Git URIs.

Testing

Check CI pipeline runs green.


  • Includes tests (if new features are introduced)
  • Commit message includes link to the ticket/issue
  • Changelog updated
  • PR open against dxw's Homebrew Tap to point the Whippet formula to the new version number

snim2 added 3 commits October 25, 2024 12:09
Previously our documentation referred to dxw endpoints
that we have used in the past. This commit updates the
docs to match our current reality.
This commit replaces our old Git endpoints which
are no longer in use with current ones in tests
and mocks.

This isn't really necessary, but if we search for old
endpoints in GitHub it's useful to avoid seeing
false-positives that we need to check manually.
@snim2 snim2 force-pushed the chore/do-not-use-old-endpoints branch from 1f6a90b to ae6671f Compare October 25, 2024 11:09
@snim2 snim2 marked this pull request as ready for review October 25, 2024 11:12
@snim2 snim2 requested a review from leedxw October 25, 2024 12:20
@snim2 snim2 merged commit 4d418c1 into main Oct 25, 2024
5 checks passed
@snim2 snim2 deleted the chore/do-not-use-old-endpoints branch October 25, 2024 17:03
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.

2 participants