Content-Length: 267869 | pFad | https://github.com/internetarchive/openlibrary/pull/9616

8E Fix debugger/debugpy not working by cdrini · Pull Request #9616 · internetarchive/openlibrary · GitHub
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

Fix debugger/debugpy not working #9616

Merged

Conversation

cdrini
Copy link
Collaborator

@cdrini cdrini commented Jul 23, 2024

Closes #9613 . The debugger now works again! 🥳

Technical

  • Update options to the latest APIs
  • Fix JS issues causing button to be disabled

Testing

Follow the steps on https://github.com/internetarchive/openlibrary/wiki/Debugging-and-Performance-Profiling#using-a-debugger

The debugger should now attach at the end of that! You should see the little floating toolbar that allows for pause/skip/etc . And you can add a break point to the code via the VS Code UI, and it'll stop there :)

Screenshot

Stakeholders

@benbdeitch

- Update options to the latest APIs
- Fix JS issues causing button to be disabled
@github-actions github-actions bot added the Priority: 2 Important, as time permits. [managed] label Jul 23, 2024
@codecov-commenter
Copy link

codecov-commenter commented Jul 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 16.21%. Comparing base (ce16a79) to head (c2037ff).
Report is 81 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #9616      +/-   ##
==========================================
+ Coverage   16.06%   16.21%   +0.15%     
==========================================
  Files          90       90              
  Lines        4769     4786      +17     
  Branches      832      829       -3     
==========================================
+ Hits          766      776      +10     
- Misses       3480     3486       +6     
- Partials      523      524       +1     

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

Copy link
Collaborator

@benbdeitch benbdeitch left a comment

Choose a reason for hiding this comment

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

I tested this out on Gitpod, and it works just fine. It's got my stamp of approval. Nice job!

@cdrini cdrini merged commit 0a43469 into internetarchive:master Jul 26, 2024
5 checks passed
@cdrini cdrini deleted the 9613/fix/debugger-not-attaching branch July 26, 2024 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: 2 Important, as time permits. [managed]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Attaching VS Code debugger no longer working
3 participants








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: https://github.com/internetarchive/openlibrary/pull/9616

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy