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

Turn tmt/utils.py into a subpackage #2982

Merged
merged 1 commit into from
Jul 12, 2024
Merged

Conversation

happz
Copy link
Collaborator

@happz happz commented Jun 3, 2024

For the code, nothing should change, as tmt.utils still exists. However, since the file is growing and growing with new stuff, I'd like to restart the initiative to split larger files to easier manageable chunks. This would be the beginning of the road for tmt.utils.

Pull Request Checklist

  • implement the feature

@happz happz added code | style Code style changes not affecting functionality code | no functional change "No Functional Change" intended. Patch should not change tmt's behavior in any way. labels Jun 3, 2024
@happz happz added this to the 1.35 milestone Jun 3, 2024
@happz happz force-pushed the split-utils-1-spawning-subpackage branch from fcee14f to 995f20a Compare June 3, 2024 08:04
@happz happz force-pushed the split-utils-1-spawning-subpackage branch from 995f20a to b8f98ee Compare June 12, 2024 20:33
@happz happz added the ci | full test Pull request is ready for the full test execution label Jun 12, 2024
@happz happz force-pushed the split-utils-1-spawning-subpackage branch from b8f98ee to ebd97c7 Compare July 6, 2024 16:38
@happz happz added the status | ready for merge The only missing piece is to do the rebase the current 'main' and let the CI finish. label Jul 9, 2024
@psss psss changed the title Turn tmt/utils.py into a subpackage Turn tmt/utils.py into a subpackage Jul 9, 2024
For the code, nothing should change, as `tmt.utils` still exists.
However, since the file is growing and growing with new stuff, I'd like
to restart the initiative to split larger files to easier manageable
chunks. This would be the beginning of the road for `tmt.utils`.
@happz happz force-pushed the split-utils-1-spawning-subpackage branch from ebd97c7 to 65eb9fb Compare July 12, 2024 07:40
@happz
Copy link
Collaborator Author

happz commented Jul 12, 2024

Unrelated failures, plenty of them. Mergin.

@happz happz merged commit 46b7a7a into main Jul 12, 2024
13 of 19 checks passed
@happz happz deleted the split-utils-1-spawning-subpackage branch July 12, 2024 11:26
The-Mule pushed a commit to The-Mule/tmt that referenced this pull request Oct 14, 2024
For the code, nothing should change, as `tmt.utils` still exists.
However, since the file is growing and growing with new stuff, I'd like
to restart the initiative to split larger files to easier manageable
chunks. This would be the beginning of the road for `tmt.utils`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci | full test Pull request is ready for the full test execution code | no functional change "No Functional Change" intended. Patch should not change tmt's behavior in any way. code | style Code style changes not affecting functionality status | ready for merge The only missing piece is to do the rebase the current 'main' and let the CI finish.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants