Skip to content

feat: add attrs support to Python preset#2183

Merged
jonaslagoni merged 6 commits intoasyncapi:masterfrom
Manancode:fix-python-attrs-preset
Mar 13, 2025
Merged

feat: add attrs support to Python preset#2183
jonaslagoni merged 6 commits intoasyncapi:masterfrom
Manancode:fix-python-attrs-preset

Conversation

@Manancode
Copy link
Contributor

This PR adds attrs support to Python preset by using @attr.s(auto_attribs=True).

Ensures proper class serialization using attrs.

@netlify
Copy link

netlify bot commented Feb 14, 2025

Deploy Preview for modelina canceled.

Name Link
🔨 Latest commit 1724434
🔍 Latest deploy log https://app.netlify.com/sites/modelina/deploys/67d20bbd46300d000889a257

@jonaslagoni jonaslagoni changed the title fix(python): add attrs support to Python preset feat: add attrs support to Python preset Feb 15, 2025
Copy link
Member

@jonaslagoni jonaslagoni left a comment

Choose a reason for hiding this comment

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

Just the tests that is missing ✌️

@Manancode
Copy link
Contributor Author

Just the tests that is missing

Sure sir will add that too

@Manancode
Copy link
Contributor Author

@jonaslagoni added the tests for both presets. Fixes issue

@coveralls
Copy link

coveralls commented Mar 2, 2025

Pull Request Test Coverage Report for Build 13822831335

Details

  • 9 of 9 (100.0%) changed or added relevant lines in 4 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.007%) to 92.404%

Totals Coverage Status
Change from base Build 13573893235: 0.007%
Covered Lines: 6487
Relevant Lines: 6882

💛 - Coveralls

Copy link
Member

@jonaslagoni jonaslagoni left a comment

Choose a reason for hiding this comment

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

@Manancode make sure you do a self-review 🙂 You need to fix some linting 🙂

@sonarqubecloud
Copy link

@jonaslagoni jonaslagoni merged commit e9f094d into asyncapi:master Mar 13, 2025
16 checks passed
@jonaslagoni
Copy link
Member

@all-contributors please add @Manancode for code, test

@allcontributors
Copy link
Contributor

@jonaslagoni

I've put up a pull request to add @Manancode! 🎉

@jonaslagoni
Copy link
Member

@Manancode I would highly recommend you make another PR that changes the docs so people can know about your new presets 🙂

@asyncapi-bot
Copy link
Contributor

🎉 This PR is included in version 4.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants