Skip to content

timezones: add#24

Open
bagder wants to merge 2 commits intomasterfrom
bagder/timezones
Open

timezones: add#24
bagder wants to merge 2 commits intomasterfrom
bagder/timezones

Conversation

@bagder
Copy link
Member

@bagder bagder commented Mar 16, 2026

image

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds the “timezones” graph to the stats build pipeline and fixes its rendering so the background map and the standard curl watermark logo can coexist.

Changes:

  • Register timezones.svg in the Makefile graph list and add build rules to generate timezones.csv and the final SVG.
  • Add the new graph to stats.sh’s output mapping list.
  • Update timezones.plot to use a non-conflicting gnuplot pixmap id for the map image.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
timezones.plot Switches the map image to pixmap 2 to avoid clobbering the shared logo watermark (pixmap 1).
stats.sh Adds the new time-zone-use-in-commits entry pointing to timezones.svg.
Makefile Includes timezones.svg in the graph set and adds rules to build the SVG and its CSV input.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@bagder bagder force-pushed the bagder/timezones branch from 775e11b to 9b29cea Compare March 16, 2026 08:33
@bagder
Copy link
Member Author

bagder commented Mar 16, 2026

Ah, the script stupidly ignores all time zones not :00 ...

@bagder
Copy link
Member Author

bagder commented Mar 16, 2026

Update:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants