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

Rename swagger_server to sdx_controller #214

Merged
merged 16 commits into from
Jan 30, 2024
Merged

Rename swagger_server to sdx_controller #214

merged 16 commits into from
Jan 30, 2024

Conversation

sajith
Copy link
Member

@sajith sajith commented Jan 26, 2024 •
edited
Loading

Resolves #213.

@sajith sajith self-assigned this Jan 26, 2024
@coveralls
Copy link

coveralls commented Jan 26, 2024 •
edited
Loading

Pull Request Test Coverage Report for Build 7701677728

  • -3 of 43 (93.02%) changed or added relevant lines in 16 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.3%) to 37.44%
Changes Missing Coverage Covered Lines Changed/Added Lines %
sdx_controller/main.py 0 3 0.0%
Totals Coverage Status
Change from base Build 7701665519: -0.3%
Covered Lines: 517
Relevant Lines: 1236
💛 - Coveralls

sajith added 10 commits January 26, 2024 17:13
@sajith
Update checkout and setup-python actions
5512f25 
Because GitHub is deprecating Node 20, and the older versions
apparently use Node 20.  Or so does the warnings say on GitHub
Actions.
@sajith
Change more paths in pyproject.toml
f12996d
@sajith
Re-enable setuptools_scm
b49260a
@sajith
Update a comment
f5aa06a
@sajith
Run isort
bd255a0
@sajith
Adjust pytest-cov path
cd33dd5
@sajith
Remove unused directory
be62954
@sajith
Rename sdx_server to sdx_controller
c7ffb8e
@sajith
Update .gitignore
a07c3af
@sajith
Replace sdx_server with sdx_controller
a5838c2 
Using:

git grep -l "sdx_server" | xargs sed -i "" -e "s/sdx_server/sdx_controller/g"
@sajith sajith changed the title Rename swagger_server to sdx_server Rename swagger_server to sdx_controller Jan 27, 2024
@sajith sajith mentioned this pull request Jan 27, 2024
Merged
@sajith sajith marked this pull request as ready for review January 29, 2024 20:09
@sajith sajith merged commit 05e3e5c into main Jan 30, 2024
9 checks passed
@sajith sajith deleted the 213.rename-swagger-server branch January 30, 2024 03:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@congwang09 congwang09 congwang09 approved these changes

@YufengXin YufengXin Awaiting requested review from YufengXin

Assignees

@sajith sajith

Labels
None yet
Projects
@sajith's project board
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Rename top-level module
3 participants
@sajith @coveralls @congwang09