Skip to content

[14.0][FIX] module_analysis: The 'code' attribute does not exist in SourceAnalysis Class - #3412

Merged
OCA-git-bot merged 1 commit into
OCA:14.0from
BinhexTeam:14.0-fix-module_analysis
Oct 8, 2025
Merged

[14.0][FIX] module_analysis: The 'code' attribute does not exist in SourceAnalysis Class#3412
OCA-git-bot merged 1 commit into
OCA:14.0from
BinhexTeam:14.0-fix-module_analysis

Conversation

@adasatorres

@adasatorres adasatorres commented Oct 8, 2025

Copy link
Copy Markdown

Hi, since Pygount doesn’t currently have a pinned version, some newer releases no longer include the 'code' attribute.I propose pinning this add-on to version 1.8.0.

@adasatorres
adasatorres force-pushed the 14.0-fix-module_analysis branch 2 times, most recently from ddabf0c to 3509e97 Compare October 8, 2025 15:23

@legalsylvain legalsylvain left a comment

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.

please don't pin the version. I don't think it's necessary.
@fcayre implementation is more flexible : #3378

@adasatorres
adasatorres force-pushed the 14.0-fix-module_analysis branch 2 times, most recently from f205892 to bfa21cf Compare October 8, 2025 15:30
@adasatorres
adasatorres force-pushed the 14.0-fix-module_analysis branch from bfa21cf to 3308a28 Compare October 8, 2025 15:31
@legalsylvain

Copy link
Copy Markdown
Contributor

Thanks !

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-3412-by-legalsylvain-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit 6a6aa2b into OCA:14.0 Oct 8, 2025
9 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at bc20ac3. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants