Skip to content

feat: Create stubs for public methods of inherited internal classes#69

Merged
lars-reimann merged 14 commits intomainfrom
64-stubs-for-public-methods
Mar 4, 2024
Merged

feat: Create stubs for public methods of inherited internal classes#69
lars-reimann merged 14 commits intomainfrom
64-stubs-for-public-methods

Conversation

@Masara
Copy link
Copy Markdown
Contributor

@Masara Masara commented Feb 28, 2024

Closes #64

Summary of Changes

Now stubs for public methods of internal classes that are being inherited will be created for the classes that inherit them.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 28, 2024

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ PYTHON black 1 0 0 0.75s
✅ PYTHON mypy 1 0 5.35s
✅ PYTHON ruff 1 0 0 0.03s
✅ REPOSITORY git_diff yes no 0.02s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.82%. Comparing base (522f38d) to head (2c626eb).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #69      +/-   ##
==========================================
+ Coverage   98.80%   98.82%   +0.01%     
==========================================
  Files          25       25              
  Lines        2098     2120      +22     
==========================================
+ Hits         2073     2095      +22     
  Misses         25       25              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Masara Masara requested a review from lars-reimann February 28, 2024 16:56
Masara added 2 commits March 1, 2024 01:47
# Conflicts:
#	tests/safeds_stubgen/stubs_generator/test_generate_stubs.py
@Masara
Copy link
Copy Markdown
Contributor Author

Masara commented Mar 1, 2024

@lars-reimann Could you check these two PRs?

@lars-reimann
Copy link
Copy Markdown
Member

Yep, tomorrow. Any preference which one to merge first? Or are they independent of each other?

@lars-reimann lars-reimann changed the title feat: Create stubs for public methods of inherited intern classes feat: Create stubs for public methods of inherited internal classes Mar 2, 2024
@Masara
Copy link
Copy Markdown
Contributor Author

Masara commented Mar 3, 2024

@lars-reimann Added all the changes, you can continue with the PR

Copy link
Copy Markdown
Member

@lars-reimann lars-reimann left a comment

Choose a reason for hiding this comment

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

Looks good! 👍

@lars-reimann lars-reimann merged commit 71b38d7 into main Mar 4, 2024
@lars-reimann lars-reimann deleted the 64-stubs-for-public-methods branch March 4, 2024 20:12
lars-reimann pushed a commit that referenced this pull request Mar 29, 2024
## [0.2.0](v0.1.0...v0.2.0) (2024-03-29)

### Features

* Added generation for Safe-DS stubs files ([#33](#33)) ([ab45b45](ab45b45))
* Correct stubs for TypeVars ([#67](#67)) ([df8c5c9](df8c5c9)), closes [#63](#63)
* Create stubs for public methods of inherited internal classes ([#69](#69)) ([71b38d7](71b38d7)), closes [#64](#64)
* Rework import generation for stubs. ([#50](#50)) ([216e179](216e179)), closes [#38](#38) [#24](#24) [#38](#38) [#24](#24)
* Safe-DS stubs also contain docstring information. ([#78](#78)) ([bdb43bd](bdb43bd))
* Stubs are created for referenced declarations in other packages ([#70](#70)) ([522f38d](522f38d)), closes [#66](#66)

### Bug Fixes

* Some packages couldn't be analyzed ([#51](#51)) ([fa3d020](fa3d020)), closes [#48](#48)
* Stub generation testing and fixing of miscellaneous bugs ([#76](#76)) ([97b0ab3](97b0ab3))
@lars-reimann
Copy link
Copy Markdown
Member

🎉 This PR is included in version 0.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@lars-reimann lars-reimann added the released Included in a release label Mar 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released Included in a release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: Generate stubs for public methods of private classes which are superclasses of public classes

3 participants