Skip to content

Conversation

@signekb
Copy link
Member

@signekb signekb commented Jul 23, 2025

Description

When I renamed the template, I forgot to rename the PackageProperties object in it to match. This PR does that.

This PR needs a quick review.

Checklist

  • Added or updated tests
  • Ran just run-all

@signekb signekb requested a review from a team as a code owner July 23, 2025 10:49
@signekb signekb changed the title test: ✅ update tests with properties obj name package_properties refactor: 🚚 rename obj in package properties script to package_properties Jul 23, 2025
@signekb signekb moved this from Todo to In Review in Iteration planning Jul 23, 2025
martonvago
martonvago previously approved these changes Jul 23, 2025
Copy link
Contributor

@martonvago martonvago left a comment

Choose a reason for hiding this comment

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

Does this need to be updated in the docs as well?

@github-project-automation github-project-automation bot moved this from In Review to In Progress in Iteration planning Jul 23, 2025
@signekb
Copy link
Member Author

signekb commented Jul 23, 2025

Yes! On it :)

lwjohnst86 and others added 2 commits July 23, 2025 13:37
`properties.py` -> `package_properties.py`; `properties` -> `package_properties` within code blocks.
@signekb signekb requested a review from martonvago July 23, 2025 11:52
@signekb signekb moved this from In Progress to In Review in Iteration planning Jul 23, 2025
@github-project-automation github-project-automation bot moved this from In Review to In Progress in Iteration planning Jul 23, 2025
@lwjohnst86 lwjohnst86 merged commit 553c30a into main Jul 23, 2025
5 checks passed
@lwjohnst86 lwjohnst86 deleted the refactor/rename-properties-obj-in-package-properties-script branch July 23, 2025 11:59
@github-project-automation github-project-automation bot moved this from In Progress to Done in Iteration planning Jul 23, 2025
lwjohnst86 pushed a commit to seedcase-project/template-data-package that referenced this pull request Jul 28, 2025
# Description

This PR renames `properties` to `package_properties` to match what was
done in seedcase-project/seedcase-sprout#1484.

This PR needs a quick review.

## Checklist

- [x] Formatted Markdown
- [x] Ran `just run-all`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants