Skip to content

change package name from - to _#290

Merged
cmhedrickREI merged 2 commits intomainfrom
5268-update-package-build
May 29, 2025
Merged

change package name from - to _#290
cmhedrickREI merged 2 commits intomainfrom
5268-update-package-build

Conversation

@cmhedrickREI
Copy link
Member

Pull Request

Related to GSA/data.gov#5268

About

Changes the - to _

PR TASKS

  • The actual code changes.
  • Tests written and passed.
  • Any changes to docs?
  • Bumped version number in setup.py (also checked on PyPi).

@cmhedrickREI cmhedrickREI requested a review from a team May 29, 2025 18:27
Copy link
Contributor

@neilmb neilmb left a comment

Choose a reason for hiding this comment

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

Looks reasonable. Are we going to have to do this to every one of our ckanext repositories now?

@cmhedrickREI
Copy link
Member Author

Looks reasonable. Are we going to have to do this to every one of our ckanext repositories now?

Thats a really good question... It seems like we may have to update most of our repos 🥲

@cmhedrickREI
Copy link
Member Author

cmhedrickREI commented May 29, 2025

@neilmb apologies as I think I jumped and looked at the sketch to quickly. Tyler had noted a commit in the ticket that is actually a change to the deploy process. That change results in the replacement of the - with an _. I went off the sketch and was wrong. I updated the sketch to mimic the change noted in the commit. The updated code should be doing what is actually expected.

@cmhedrickREI
Copy link
Member Author

(Screencap of the current geodata package showing the - and the datagovtheme showing the _ in the distribution names)
Screenshot from 2025-05-29 16-32-47

(Screencap of after following the new build process copied from the datagovtheme, resulting in the file with _)
Screenshot from 2025-05-29 16-32-23

@cmhedrickREI cmhedrickREI merged commit 139e6c6 into main May 29, 2025
5 checks passed
@cmhedrickREI cmhedrickREI deleted the 5268-update-package-build branch May 29, 2025 20:47
@cmhedrickREI
Copy link
Member Author

(Screencap of successful file name change on Pypi)
Screenshot from 2025-05-30 06-20-42

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