Skip to content

fix: rename policy packages to now policy#70

Merged
Marc-André Moreau (mamoreau-devolutions) merged 2 commits into
masterfrom
rename-again
Jun 22, 2026
Merged

fix: rename policy packages to now policy#70
Marc-André Moreau (mamoreau-devolutions) merged 2 commits into
masterfrom
rename-again

Conversation

@mamoreau-devolutions

Copy link
Copy Markdown
Contributor

Summary

  • rename the .NET policy package/project/namespace to Devolutions.NowPolicy
  • rename the Rust policy crate to now-policy and update crate imports/generator binary
  • update CI, xtask paths, Cargo.lock, and policy docs for the new naming
  • keep policy wire-format identifiers unchanged (PackageBrokerPolicy and broker-policy schema URI)

Validation

  • cargo test -p now-policy
  • dotnet test policies/dotnet/Devolutions.NowPolicy.slnx
  • dotnet format policies/dotnet/Devolutions.NowPolicy.slnx --verify-no-changes
  • git diff --cached --check

@vnikonov-devolutions

Copy link
Copy Markdown
Contributor

Should we also rename schema url?

https://devolutions.net/schemas/broker-policy.schema.1.0.json
->
https://devolutions.net/schemas/now-policy.schema.1.0.json

@mamoreau-devolutions

Copy link
Copy Markdown
Contributor Author

Updated for Vlad's schema URL feedback: renamed the schema URL to https://devolutions.net/schemas/now-policy.schema.1.0.json, renamed the generated schema file to devolutions.now-policy.schema.json, updated samples/tests/constants, and regenerated the schema. Validation: cargo test -p now-policy, dotnet test policies/dotnet/Devolutions.NowPolicy.slnx, dotnet format policies/dotnet/Devolutions.NowPolicy.slnx --verify-no-changes, cargo fmt --check -p now-policy.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@CBenoit Benoît Cortier (CBenoit) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@mamoreau-devolutions Marc-André Moreau (mamoreau-devolutions) merged commit ac4a313 into master Jun 22, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants