Sitelet https://web.archive.org/web/20200915024139/https://github.com/cs01/gdbgui/pull/350
Skip to content

/ gdbgui

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

refactor backend #350

Merged
merged 14 commits into from Aug 24, 2020
Merged

refactor backend #350

merged 14 commits into from Aug 24, 2020

Conversation

@cs01
Copy link
Owner

cs01 commented Aug 24, 2020 •
edited

  • I have added an entry to docs/changelog.md

Summary of changes

  • Refactor backend code
  • Break into multiple modules
  • Hide repetitive "no registers" message in gdbgui browser console
  • Add more unit tests

Test plan

Tested by running

gdbgui --debug
nox -s tests-3.7
cs01 added 14 commits Aug 23, 2020
@cs01
rename backend to cli, add constants file
bdd2b39
@cs01
move app to module
2df45be
@cs01
remove some dead code; move functions to util
596c39b
@cs01
more util functions and move read file route
152af62
@cs01
move more routes; fix some lint errors
47c8170
@cs01
move main route
1a618f3
@cs01
move all routes from cli
e307732
@cs01
move function and update highlight logic
abe3ead
@cs01
fix lint
906db78
@cs01
move websocket routes out of cli file
d8d22d5
@cs01
update changelog, increment version
Loading status checks…
8f9ef43
@cs01
fix __main__
Loading status checks…
7aeb389
@cs01
fix tests and build
Loading status checks…
41e7d69
@cs01
add more tests
Loading status checks…
db10204
@cs01 cs01 merged commit 853f141 into master Aug 24, 2020
7 checks passed
7 checks passed
build (ubuntu-latest, 3.8)
Details
run_tests (ubuntu-latest, 3.6)
Details
build (macos-latest, 3.8)
Details
run_tests (ubuntu-latest, 3.7)
Details
run_tests (ubuntu-latest, 3.8)
Details
lint
Details
docs
Details
@cs01 cs01 deleted the cs01/refactor-backend branch Aug 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
@cs01
You can’t perform that action at this time.