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.

Sign up for GitHub

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

Jump to bottom

Move static IOMarket methods to namespace #4690

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Move static IOMarket methods to namespace #4690

wants to merge 3 commits into from

Conversation

ramon-bernardo
Copy link
Contributor

@ramon-bernardo ramon-bernardo commented May 19, 2024 •
edited

Pull Request Prelude

Changes Proposed

  • Bring closure to the IOMarket class and its static instance by moving it to a namespace.

IOMarket::x to tfs::io::market::x

@ramon-bernardo
Copy link
Contributor Author

ramon-bernardo commented May 26, 2024 •
edited

tfs::db::market it's more readable?

@EvilHero90 EvilHero90 added the enhancement Increase or improvement in quality, value, or extent label Jun 2, 2024
@EvilHero90 EvilHero90 added this to the 1.8 milestone Jun 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DSpeichert DSpeichert Awaiting requested review from DSpeichert

@ranisalt ranisalt Awaiting requested review from ranisalt

@EvilHero90 EvilHero90 Awaiting requested review from EvilHero90

@nekiro nekiro Awaiting requested review from nekiro

@MillhioreBT MillhioreBT Awaiting requested review from MillhioreBT

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
enhancement Increase or improvement in quality, value, or extent
Projects
TFS 1.8
Status: No status
Milestone
1.8
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@ramon-bernardo @EvilHero90