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

(maint) pdk 340 update #555

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

(maint) pdk 340 update #555

wants to merge 2 commits into from

Conversation

gavindidrichsen
Copy link

Update PDK to 3.4.0

Summary

This PR updates the module from PDK 3.0.0 to PDK 3.4.0 with minimal, non-intrusive changes. The update includes necessary fixes for Ruby 3.2.5 compatibility, particularly around YAML symbol parsing.

Changes

  • Fixed YAML parsing issues by properly quoting Ruby symbols in .sync.yml
  • Updated PDK version from 3.0.0 to 3.4.0
  • Updated gem dependencies for Ruby 3.x compatibility
  • Updated CI workflow configurations
  • Added .vscode to .gitignore to prevent IDE-specific files from being tracked

Testing

  • All unit tests passing: 100 examples, 0 failures, 6 pending (expected)
  • No functional changes to module code
  • No changes to module behavior or interface

Note

The 6 pending tests are expected and are related to Bolt functionality that requires special handling in the test environment. These were present before this update and are not affected by these changes.

Due to stricter YAML parsing rules in Ruby 3.2.5 (used by PDK 3.4.0),
unquoted Ruby symbols in .sync.yml cause parsing errors. This updates
all Ruby symbols to be properly quoted strings (e.g. ':development'
instead of :development).

The error encountered was:

```bash
Psych::DisallowedClass: Tried to load unspecified class: Symbol
  from psych/class_loader.rb:99:in `find'
```

This change allows the module to be properly updated using PDK 3.4.0
while maintaining the same functionality.

Signed-off-by: Gavin Didrichsen <gavin.didrichsen@gmail.com>
@gavindidrichsen gavindidrichsen force-pushed the pdk_340_update branch 2 times, most recently from 83bf28e to 075d946 Compare February 28, 2025 01:10
Updates the module to PDK 3.4.0 with minimal changes:
- Updates gem dependencies for Ruby 3.x compatibility
- Updates CI workflow configuration
- All unit tests passing with 6 expected pending tests
- No functional changes to module code
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.

1 participant