Skip to content

Update the uploading command in the developer documentation#4752

Merged
julienw merged 1 commit intofirefox-devtools:mainfrom
julienw:change-command-in-doc
Sep 26, 2023
Merged

Update the uploading command in the developer documentation#4752
julienw merged 1 commit intofirefox-devtools:mainfrom
julienw:change-command-in-doc

Conversation

@julienw
Copy link
Contributor

@julienw julienw commented Sep 19, 2023

I've seen this while looking at other doc, and decided to update it. I tested the command locally, it worked for me.

@julienw julienw requested a review from canova September 19, 2023 12:48
@codecov
Copy link

codecov bot commented Sep 19, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (2a8c44b) 88.34% compared to head (0e196c7) 88.34%.
Report is 5 commits behind head on main.

❗ Current head 0e196c7 differs from pull request most recent head 9137a66. Consider uploading reports for the commit 9137a66 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4752   +/-   ##
=======================================
  Coverage   88.34%   88.34%           
=======================================
  Files         300      300           
  Lines       26781    26781           
  Branches     7224     7224           
=======================================
  Hits        23659    23659           
  Misses       2910     2910           
  Partials      212      212           

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

Copy link
Member

@canova canova left a comment

Choose a reason for hiding this comment

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

I don't mind landing it like this, but I'm not sure about referencing a relative file (tools/decode_jwt_payload.py) in a bash function 🤔 But also can't think of a better solution :)

@julienw julienw force-pushed the change-command-in-doc branch from 0e196c7 to 9137a66 Compare September 26, 2023 11:41
@julienw julienw enabled auto-merge (squash) September 26, 2023 13:42
@julienw julienw force-pushed the change-command-in-doc branch from 9137a66 to 367f984 Compare September 26, 2023 13:42
@julienw julienw merged commit 1a755fb into firefox-devtools:main Sep 26, 2023
tenderlove added a commit to tenderlove/profiler that referenced this pull request Jan 16, 2024
* main: (37 commits)
  Show the hovered time in the ruler at the top of the timeline. (PR firefox-devtools#4748)
  Hardcode 'project' to 'firefox-profiler' in .taskcluster.yml (PR firefox-devtools#4759)
  Update all Yarn dependencies (2023-09-27) (PR firefox-devtools#4758)
  Update the uploading command in the developer documentation (PR firefox-devtools#4752)
  Make prettier ignore the taskcluster files
  Add taskcluster
  Update all Yarn dependencies (2023-09-20) (PR firefox-devtools#4756)
  Add a robots.txt file to disallow indexing our profile links (PR firefox-devtools#4753)
  Avoid showing the calltree panel in profiles without samples. (PR firefox-devtools#4744)
  Display the vertical line in the timeline when hovering the marker chart or stack chart (PR firefox-devtools#4742)
  Draw a single rectangle for 2 markers in bar charts only if both the start and end horizontal pixels match. (PR firefox-devtools#4734)
  Update all Yarn dependencies (2023-09-13) (firefox-devtools#4746)
  Update all Yarn dependencies (2023-09-06) (PR firefox-devtools#4741)
  Remove prefer-wait-for rule in our config because it's been removed from the plugin
  ⬆️ Update eslint-plugin-testing-library to version 6.0.1
  Use a cache for prettier (PR firefox-devtools#4731)
  Remove leftover eslint error
  Run prettier on the codebase
  Upgrade prettier to latest version
  Ignore the coverage directory when present
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants