Skip to content

Use https in license headers - #6039

Merged
bubblobill merged 3 commits into
RPTools:developfrom
kwvanderlinde:bugfix/1391-http-to-https-license
Sep 24, 2026
Merged

bubblobill merged 3 commits into
RPTools:developfrom
kwvanderlinde:bugfix/1391-http-to-https-license

Conversation

@kwvanderlinde

@kwvanderlinde kwvanderlinde commented Aug 20, 2026 •

Copy link
Copy Markdown
Collaborator

Identify the Bug or Feature request

Improves #1391

Description of the Change

Changes the links in spotless.license.java to use https: instead of http:, and pushes those changes into each file.

Also updates a couple of outliers to use the standard license text (MathFunctions, net.rptools.maptool.client.script.javascript.api.package-info.java).

I kept this PR separate from #6038 since it is purely non-functional text and is essentially the same change repeated over a couple thousand files.

Possible Drawbacks

None

Documentation Notes

N/A

Release Notes

N/A


This change is Reviewable

@kwvanderlinde
kwvanderlinde force-pushed the bugfix/1391-http-to-https-license branch from 5132b46 to 5621f11 Compare August 20, 2026 23:55
@kwvanderlinde
kwvanderlinde force-pushed the bugfix/1391-http-to-https-license branch from 5621f11 to 1b097c8 Compare August 20, 2026 23:56
@kwvanderlinde kwvanderlinde moved this from Todo to Awaiting-Review in MapTool 1.19 Aug 20, 2026

@bubblobill bubblobill left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

No obvious errors

@github-project-automation github-project-automation Bot moved this from Awaiting-Review to To-Be-Merged in MapTool 1.19 Sep 6, 2026
@bubblobill
bubblobill added this pull request to the merge queue Sep 24, 2026
Merged via the queue into RPTools:develop with commit 7865fa8 Sep 24, 2026
5 checks passed
@github-project-automation github-project-automation Bot moved this from To-Be-Merged to Merged in MapTool 1.19 Sep 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

2 participants