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: Temp suffix & 5v marker #448

Merged
merged 1 commit into from
Nov 4, 2024
Merged

Conversation

mrv777
Copy link
Contributor

@mrv777 mrv777 commented Nov 3, 2024

Description:

  • Show current suffix for temp points on the chart and add the suffix to the axes.
  • Make the average hashrate a little easier to see
  • Add a 5.0v marker on the input voltage progress bar

@skot
Copy link
Owner

skot commented Nov 4, 2024

I got the following warning from node.js when compiling... I don't think we had this before?

(node:69549) [DEP0147] DeprecationWarning: In future versions of Node.js, fs.rmdir(path, { recursive: true }) will be removed. Use fs.rm(path, { recursive: true }) instead
(Use `node --trace-deprecation ...` to show where the warning was created)

I'm running Node v20.18.0

@skot
Copy link
Owner

skot commented Nov 4, 2024

looking good!
image

@mrv777
Copy link
Contributor Author

mrv777 commented Nov 4, 2024

I got the following warning from node.js when compiling... I don't think we had this before?

(node:69549) [DEP0147] DeprecationWarning: In future versions of Node.js, fs.rmdir(path, { recursive: true }) will be removed. Use fs.rm(path, { recursive: true }) instead
(Use `node --trace-deprecation ...` to show where the warning was created)

That seems to be coming from the script: only-gzip.js

@skot
Copy link
Owner

skot commented Nov 4, 2024

I got the following warning from node.js when compiling... I don't think we had this before?

(node:69549) [DEP0147] DeprecationWarning: In future versions of Node.js, fs.rmdir(path, { recursive: true }) will be removed. Use fs.rm(path, { recursive: true }) instead
(Use `node --trace-deprecation ...` to show where the warning was created)

That seems to be coming from the script: only-gzip.js

Ah, ok. I see it happens on master too. I'll figure out where that came from.

@skot skot merged commit b5d0ea5 into skot:master Nov 4, 2024
1 check passed
benjamin-wilson pushed a commit that referenced this pull request Nov 16, 2024
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