chore: release 9.4.1 - #2878

Closed
github-actions[bot] wants to merge 1 commit into
mainfrom
release-v9.4.1
Closed

chore: release 9.4.1#2878
github-actions[bot] wants to merge 1 commit into
mainfrom
release-v9.4.1

Conversation

@github-actions

@github-actionsgithub-actionsBot commented Jun 22, 2023

Copy link
Copy Markdown
Contributor

🤖 I have created a release *beep* *boop*

9.4.1 (2023-10-27)

Bug Fixes

  • create Python symlink only during builds, and clean it up after (#2721) (0f1f667)

Core

  • find python checks order changed on windows (#2872) (b030555)

Tests

Doc

  • Add note about Python symlinks (PR 2362) to CHANGELOG.md for 9.1.0 (#2783) (b3d41ae)
  • README.md Do not hardcode the supported versions of Python (#2880) (bb93b94)
  • Update windows installation instructions in README.md (#2882) (c9caa2e)

This PR was generated with Release Please. See documentation.

@lukekarrys

Copy link
Copy Markdown
Member

This shouldn't be released in it's current state. #2770 landed with a breaking change but isn't formatted like a conventional commit, so it didn't trigger this PR to be 10.0.0 as it should be.

@legobeat

legobeat commented Jul 5, 2023

Copy link
Copy Markdown

@lukekarrys Do you think we can consider #2849 for inclusion into a 9.4.1? Given that #2796 got included in 9.4.0 and is actually breaking Node 12.x support (#2873).

Then a more long-term solution which doesn't need to consider 12.x support can be targeting a 10.x release.

@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from ebae5d2 to a43758dCompareJuly 20, 2023 13:05
@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from ee68a4f to 5c042eeCompareAugust 26, 2023 08:48
@Trott

Copy link
Copy Markdown
Member

If someone does git commit --allow-empty -m "chore: release 10.0.0" -m "Release-As: 10.0.0" and pushes that to the main branch, I think Release Please will either update this PR or open a new one to release this stuff as 10.0.0. I think.....

@mfranzs

Copy link
Copy Markdown

Hi - is there any documentation on release processes here / when we can hope the latest main will be released? There are some critical fixes breaking Electron builds that we're waiting for. Thank you for this package!

@lukekarrys

Copy link
Copy Markdown
Member

There is not any current documentation @mfranzs. But my goal is to release the latest main branch this week. Although I should note that it will likely be release as a new major version 10.0.0.

@mfranzs

Copy link
Copy Markdown

Thank you very much!

@legobeat

legobeat commented Oct 3, 2023

Copy link
Copy Markdown

@lukekarrys any chance we can actually get a 9.x release? LMK if you see any blockers or other things you would need assistance with to make that happen.

^9.x is still dependended on by various unmaintained widely used ecosystem packages..

@DeeDeeG

DeeDeeG commented Oct 3, 2023

Copy link
Copy Markdown
Contributor

Stuff to consider doing before releasing?

EDIT: Besides @legobeat's point above (which I don't want to bury with my comment so shortly after)... Might want to resolve concerns mentioned in #2857 (comment) before releasing?

This bit:

Seems like the check [OP linked to in their comment] should be !=, not <. Otherwise changes requiring bumping installVersion have to be backwards-compatible due to the aforementioned scenario, which doesn't seem reasonable. If the installVersion do not match it seems reasonable to force a reinstall.

Would be good not to impose backward-compatibility guarantees on any stuff that depends on a particular installVersion matching, I think? And it's a ~3 character diff to fix it. I can do a PR going off of that person's comment if wanted. (Or the folks from that PR might have input if they were pinged, I guess.)


How to release / How to set the semver for the release

And (at least for stuff that happens on GitHub) the release process is basically https://github.com/google-github-actions/release-please-action#readme --> https://github.com/googleapis/release-please#readme. (And after that, I'm sure some maintainer(s) here know about how publishing the new version to the npm package registry works.)

You can manually set the version number (like 9.4.1 or 10.0.0) for release-please to follow by adding a commit with Release-As: x.x.x in the body of the commit message (below the first line)
https://github.com/googleapis/release-please#how-do-i-change-the-version-number

I did the PR to add release-please to this repo at the request of multiple active node-gyp contributors at the time, so if I can be of help by answering questions why it was set up any particular way I am happy to do so, but it was set up kind of arbitrarily just to make releases more automated than the existing process, to put it in a nutshell.

@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from 5d0471f to e62ac4fCompareOctober 27, 2023 03:43
@lukekarrys

Copy link
Copy Markdown
Member

I'm going to close this release PR so that when #2917 lands in release/v9, the new release-please action for that branch will create this v9.4.1 PR. And then I will push a new commit (as mentioned above #2878 (comment)) to create a v10.0.0 PR.

We can also use the new v10.0.0 PR to discuss any other changes that should land before the semver major is released.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@lukekarrys@legobeat@Trott@mfranzs@DeeDeeG
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content

chore: release 9.4.1 - #2878

Closed
github-actions[bot] wants to merge 1 commit into
mainfrom
release-v9.4.1
Closed

chore: release 9.4.1#2878
github-actions[bot] wants to merge 1 commit into
mainfrom
release-v9.4.1

Conversation

@github-actions

@github-actionsgithub-actionsBot commented Jun 22, 2023

Copy link
Copy Markdown
Contributor

🤖 I have created a release *beep* *boop*

9.4.1 (2023-10-27)

Bug Fixes

  • create Python symlink only during builds, and clean it up after (#2721) (0f1f667)

Core

  • find python checks order changed on windows (#2872) (b030555)

Tests

Doc

  • Add note about Python symlinks (PR 2362) to CHANGELOG.md for 9.1.0 (#2783) (b3d41ae)
  • README.md Do not hardcode the supported versions of Python (#2880) (bb93b94)
  • Update windows installation instructions in README.md (#2882) (c9caa2e)

This PR was generated with Release Please. See documentation.

@lukekarrys

Copy link
Copy Markdown
Member

This shouldn't be released in it's current state. #2770 landed with a breaking change but isn't formatted like a conventional commit, so it didn't trigger this PR to be 10.0.0 as it should be.

@legobeat

legobeat commented Jul 5, 2023

Copy link
Copy Markdown

@lukekarrys Do you think we can consider #2849 for inclusion into a 9.4.1? Given that #2796 got included in 9.4.0 and is actually breaking Node 12.x support (#2873).

Then a more long-term solution which doesn't need to consider 12.x support can be targeting a 10.x release.

@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from ebae5d2 to a43758dCompareJuly 20, 2023 13:05
@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from ee68a4f to 5c042eeCompareAugust 26, 2023 08:48
@Trott

Copy link
Copy Markdown
Member

If someone does git commit --allow-empty -m "chore: release 10.0.0" -m "Release-As: 10.0.0" and pushes that to the main branch, I think Release Please will either update this PR or open a new one to release this stuff as 10.0.0. I think.....

@mfranzs

Copy link
Copy Markdown

Hi - is there any documentation on release processes here / when we can hope the latest main will be released? There are some critical fixes breaking Electron builds that we're waiting for. Thank you for this package!

@lukekarrys

Copy link
Copy Markdown
Member

There is not any current documentation @mfranzs. But my goal is to release the latest main branch this week. Although I should note that it will likely be release as a new major version 10.0.0.

@mfranzs

Copy link
Copy Markdown

Thank you very much!

@legobeat

legobeat commented Oct 3, 2023

Copy link
Copy Markdown

@lukekarrys any chance we can actually get a 9.x release? LMK if you see any blockers or other things you would need assistance with to make that happen.

^9.x is still dependended on by various unmaintained widely used ecosystem packages..

@DeeDeeG

DeeDeeG commented Oct 3, 2023

Copy link
Copy Markdown
Contributor

Stuff to consider doing before releasing?

EDIT: Besides @legobeat's point above (which I don't want to bury with my comment so shortly after)... Might want to resolve concerns mentioned in #2857 (comment) before releasing?

This bit:

Seems like the check [OP linked to in their comment] should be !=, not <. Otherwise changes requiring bumping installVersion have to be backwards-compatible due to the aforementioned scenario, which doesn't seem reasonable. If the installVersion do not match it seems reasonable to force a reinstall.

Would be good not to impose backward-compatibility guarantees on any stuff that depends on a particular installVersion matching, I think? And it's a ~3 character diff to fix it. I can do a PR going off of that person's comment if wanted. (Or the folks from that PR might have input if they were pinged, I guess.)


How to release / How to set the semver for the release

And (at least for stuff that happens on GitHub) the release process is basically https://github.com/google-github-actions/release-please-action#readme --> https://github.com/googleapis/release-please#readme. (And after that, I'm sure some maintainer(s) here know about how publishing the new version to the npm package registry works.)

You can manually set the version number (like 9.4.1 or 10.0.0) for release-please to follow by adding a commit with Release-As: x.x.x in the body of the commit message (below the first line)
https://github.com/googleapis/release-please#how-do-i-change-the-version-number

I did the PR to add release-please to this repo at the request of multiple active node-gyp contributors at the time, so if I can be of help by answering questions why it was set up any particular way I am happy to do so, but it was set up kind of arbitrarily just to make releases more automated than the existing process, to put it in a nutshell.

@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from 5d0471f to e62ac4fCompareOctober 27, 2023 03:43
@lukekarrys

Copy link
Copy Markdown
Member

I'm going to close this release PR so that when #2917 lands in release/v9, the new release-please action for that branch will create this v9.4.1 PR. And then I will push a new commit (as mentioned above #2878 (comment)) to create a v10.0.0 PR.

We can also use the new v10.0.0 PR to discuss any other changes that should land before the semver major is released.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@lukekarrys@legobeat@Trott@mfranzs@DeeDeeG
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

chore: release 9.4.1 - #2878

Closed
github-actions[bot] wants to merge 1 commit into
mainfrom
release-v9.4.1
Closed

chore: release 9.4.1#2878
github-actions[bot] wants to merge 1 commit into
mainfrom
release-v9.4.1

Conversation

@github-actions

@github-actionsgithub-actionsBot commented Jun 22, 2023

Copy link
Copy Markdown
Contributor

🤖 I have created a release *beep* *boop*

9.4.1 (2023-10-27)

Bug Fixes

  • create Python symlink only during builds, and clean it up after (#2721) (0f1f667)

Core

  • find python checks order changed on windows (#2872) (b030555)

Tests

Doc

  • Add note about Python symlinks (PR 2362) to CHANGELOG.md for 9.1.0 (#2783) (b3d41ae)
  • README.md Do not hardcode the supported versions of Python (#2880) (bb93b94)
  • Update windows installation instructions in README.md (#2882) (c9caa2e)

This PR was generated with Release Please. See documentation.

@lukekarrys

Copy link
Copy Markdown
Member

This shouldn't be released in it's current state. #2770 landed with a breaking change but isn't formatted like a conventional commit, so it didn't trigger this PR to be 10.0.0 as it should be.

@legobeat

legobeat commented Jul 5, 2023

Copy link
Copy Markdown

@lukekarrys Do you think we can consider #2849 for inclusion into a 9.4.1? Given that #2796 got included in 9.4.0 and is actually breaking Node 12.x support (#2873).

Then a more long-term solution which doesn't need to consider 12.x support can be targeting a 10.x release.

@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from ebae5d2 to a43758dCompareJuly 20, 2023 13:05
@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from ee68a4f to 5c042eeCompareAugust 26, 2023 08:48
@Trott

Copy link
Copy Markdown
Member

If someone does git commit --allow-empty -m "chore: release 10.0.0" -m "Release-As: 10.0.0" and pushes that to the main branch, I think Release Please will either update this PR or open a new one to release this stuff as 10.0.0. I think.....

@mfranzs

Copy link
Copy Markdown

Hi - is there any documentation on release processes here / when we can hope the latest main will be released? There are some critical fixes breaking Electron builds that we're waiting for. Thank you for this package!

@lukekarrys

Copy link
Copy Markdown
Member

There is not any current documentation @mfranzs. But my goal is to release the latest main branch this week. Although I should note that it will likely be release as a new major version 10.0.0.

@mfranzs

Copy link
Copy Markdown

Thank you very much!

@legobeat

legobeat commented Oct 3, 2023

Copy link
Copy Markdown

@lukekarrys any chance we can actually get a 9.x release? LMK if you see any blockers or other things you would need assistance with to make that happen.

^9.x is still dependended on by various unmaintained widely used ecosystem packages..

@DeeDeeG

DeeDeeG commented Oct 3, 2023

Copy link
Copy Markdown
Contributor

Stuff to consider doing before releasing?

EDIT: Besides @legobeat's point above (which I don't want to bury with my comment so shortly after)... Might want to resolve concerns mentioned in #2857 (comment) before releasing?

This bit:

Seems like the check [OP linked to in their comment] should be !=, not <. Otherwise changes requiring bumping installVersion have to be backwards-compatible due to the aforementioned scenario, which doesn't seem reasonable. If the installVersion do not match it seems reasonable to force a reinstall.

Would be good not to impose backward-compatibility guarantees on any stuff that depends on a particular installVersion matching, I think? And it's a ~3 character diff to fix it. I can do a PR going off of that person's comment if wanted. (Or the folks from that PR might have input if they were pinged, I guess.)


How to release / How to set the semver for the release

And (at least for stuff that happens on GitHub) the release process is basically https://github.com/google-github-actions/release-please-action#readme --> https://github.com/googleapis/release-please#readme. (And after that, I'm sure some maintainer(s) here know about how publishing the new version to the npm package registry works.)

You can manually set the version number (like 9.4.1 or 10.0.0) for release-please to follow by adding a commit with Release-As: x.x.x in the body of the commit message (below the first line)
https://github.com/googleapis/release-please#how-do-i-change-the-version-number

I did the PR to add release-please to this repo at the request of multiple active node-gyp contributors at the time, so if I can be of help by answering questions why it was set up any particular way I am happy to do so, but it was set up kind of arbitrarily just to make releases more automated than the existing process, to put it in a nutshell.

@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from 5d0471f to e62ac4fCompareOctober 27, 2023 03:43
@lukekarrys

Copy link
Copy Markdown
Member

I'm going to close this release PR so that when #2917 lands in release/v9, the new release-please action for that branch will create this v9.4.1 PR. And then I will push a new commit (as mentioned above #2878 (comment)) to create a v10.0.0 PR.

We can also use the new v10.0.0 PR to discuss any other changes that should land before the semver major is released.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@lukekarrys@legobeat@Trott@mfranzs@DeeDeeG
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

chore: release 9.4.1 - #2878

Closed
github-actions[bot] wants to merge 1 commit into
mainfrom
release-v9.4.1
Closed

chore: release 9.4.1#2878
github-actions[bot] wants to merge 1 commit into
mainfrom
release-v9.4.1

Conversation

@github-actions

@github-actionsgithub-actionsBot commented Jun 22, 2023

Copy link
Copy Markdown
Contributor

🤖 I have created a release *beep* *boop*

9.4.1 (2023-10-27)

Bug Fixes

  • create Python symlink only during builds, and clean it up after (#2721) (0f1f667)

Core

  • find python checks order changed on windows (#2872) (b030555)

Tests

Doc

  • Add note about Python symlinks (PR 2362) to CHANGELOG.md for 9.1.0 (#2783) (b3d41ae)
  • README.md Do not hardcode the supported versions of Python (#2880) (bb93b94)
  • Update windows installation instructions in README.md (#2882) (c9caa2e)

This PR was generated with Release Please. See documentation.

@lukekarrys

Copy link
Copy Markdown
Member

This shouldn't be released in it's current state. #2770 landed with a breaking change but isn't formatted like a conventional commit, so it didn't trigger this PR to be 10.0.0 as it should be.

@legobeat

legobeat commented Jul 5, 2023

Copy link
Copy Markdown

@lukekarrys Do you think we can consider #2849 for inclusion into a 9.4.1? Given that #2796 got included in 9.4.0 and is actually breaking Node 12.x support (#2873).

Then a more long-term solution which doesn't need to consider 12.x support can be targeting a 10.x release.

@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from ebae5d2 to a43758dCompareJuly 20, 2023 13:05
@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from ee68a4f to 5c042eeCompareAugust 26, 2023 08:48
@Trott

Copy link
Copy Markdown
Member

If someone does git commit --allow-empty -m "chore: release 10.0.0" -m "Release-As: 10.0.0" and pushes that to the main branch, I think Release Please will either update this PR or open a new one to release this stuff as 10.0.0. I think.....

@mfranzs

Copy link
Copy Markdown

Hi - is there any documentation on release processes here / when we can hope the latest main will be released? There are some critical fixes breaking Electron builds that we're waiting for. Thank you for this package!

@lukekarrys

Copy link
Copy Markdown
Member

There is not any current documentation @mfranzs. But my goal is to release the latest main branch this week. Although I should note that it will likely be release as a new major version 10.0.0.

@mfranzs

Copy link
Copy Markdown

Thank you very much!

@legobeat

legobeat commented Oct 3, 2023

Copy link
Copy Markdown

@lukekarrys any chance we can actually get a 9.x release? LMK if you see any blockers or other things you would need assistance with to make that happen.

^9.x is still dependended on by various unmaintained widely used ecosystem packages..

@DeeDeeG

DeeDeeG commented Oct 3, 2023

Copy link
Copy Markdown
Contributor

Stuff to consider doing before releasing?

EDIT: Besides @legobeat's point above (which I don't want to bury with my comment so shortly after)... Might want to resolve concerns mentioned in #2857 (comment) before releasing?

This bit:

Seems like the check [OP linked to in their comment] should be !=, not <. Otherwise changes requiring bumping installVersion have to be backwards-compatible due to the aforementioned scenario, which doesn't seem reasonable. If the installVersion do not match it seems reasonable to force a reinstall.

Would be good not to impose backward-compatibility guarantees on any stuff that depends on a particular installVersion matching, I think? And it's a ~3 character diff to fix it. I can do a PR going off of that person's comment if wanted. (Or the folks from that PR might have input if they were pinged, I guess.)


How to release / How to set the semver for the release

And (at least for stuff that happens on GitHub) the release process is basically https://github.com/google-github-actions/release-please-action#readme --> https://github.com/googleapis/release-please#readme. (And after that, I'm sure some maintainer(s) here know about how publishing the new version to the npm package registry works.)

You can manually set the version number (like 9.4.1 or 10.0.0) for release-please to follow by adding a commit with Release-As: x.x.x in the body of the commit message (below the first line)
https://github.com/googleapis/release-please#how-do-i-change-the-version-number

I did the PR to add release-please to this repo at the request of multiple active node-gyp contributors at the time, so if I can be of help by answering questions why it was set up any particular way I am happy to do so, but it was set up kind of arbitrarily just to make releases more automated than the existing process, to put it in a nutshell.

@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from 5d0471f to e62ac4fCompareOctober 27, 2023 03:43
@lukekarrys

Copy link
Copy Markdown
Member

I'm going to close this release PR so that when #2917 lands in release/v9, the new release-please action for that branch will create this v9.4.1 PR. And then I will push a new commit (as mentioned above #2878 (comment)) to create a v10.0.0 PR.

We can also use the new v10.0.0 PR to discuss any other changes that should land before the semver major is released.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@lukekarrys@legobeat@Trott@mfranzs@DeeDeeG
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content

chore: release 9.4.1 - #2878

Closed
github-actions[bot] wants to merge 1 commit into
mainfrom
release-v9.4.1
Closed

chore: release 9.4.1#2878
github-actions[bot] wants to merge 1 commit into
mainfrom
release-v9.4.1

Conversation

@github-actions

@github-actionsgithub-actionsBot commented Jun 22, 2023

Copy link
Copy Markdown
Contributor

🤖 I have created a release *beep* *boop*

9.4.1 (2023-10-27)

Bug Fixes

  • create Python symlink only during builds, and clean it up after (#2721) (0f1f667)

Core

  • find python checks order changed on windows (#2872) (b030555)

Tests

Doc

  • Add note about Python symlinks (PR 2362) to CHANGELOG.md for 9.1.0 (#2783) (b3d41ae)
  • README.md Do not hardcode the supported versions of Python (#2880) (bb93b94)
  • Update windows installation instructions in README.md (#2882) (c9caa2e)

This PR was generated with Release Please. See documentation.

@lukekarrys

Copy link
Copy Markdown
Member

This shouldn't be released in it's current state. #2770 landed with a breaking change but isn't formatted like a conventional commit, so it didn't trigger this PR to be 10.0.0 as it should be.

@legobeat

legobeat commented Jul 5, 2023

Copy link
Copy Markdown

@lukekarrys Do you think we can consider #2849 for inclusion into a 9.4.1? Given that #2796 got included in 9.4.0 and is actually breaking Node 12.x support (#2873).

Then a more long-term solution which doesn't need to consider 12.x support can be targeting a 10.x release.

@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from ebae5d2 to a43758dCompareJuly 20, 2023 13:05
@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from ee68a4f to 5c042eeCompareAugust 26, 2023 08:48
@Trott

Copy link
Copy Markdown
Member

If someone does git commit --allow-empty -m "chore: release 10.0.0" -m "Release-As: 10.0.0" and pushes that to the main branch, I think Release Please will either update this PR or open a new one to release this stuff as 10.0.0. I think.....

@mfranzs

Copy link
Copy Markdown

Hi - is there any documentation on release processes here / when we can hope the latest main will be released? There are some critical fixes breaking Electron builds that we're waiting for. Thank you for this package!

@lukekarrys

Copy link
Copy Markdown
Member

There is not any current documentation @mfranzs. But my goal is to release the latest main branch this week. Although I should note that it will likely be release as a new major version 10.0.0.

@mfranzs

Copy link
Copy Markdown

Thank you very much!

@legobeat

legobeat commented Oct 3, 2023

Copy link
Copy Markdown

@lukekarrys any chance we can actually get a 9.x release? LMK if you see any blockers or other things you would need assistance with to make that happen.

^9.x is still dependended on by various unmaintained widely used ecosystem packages..

@DeeDeeG

DeeDeeG commented Oct 3, 2023

Copy link
Copy Markdown
Contributor

Stuff to consider doing before releasing?

EDIT: Besides @legobeat's point above (which I don't want to bury with my comment so shortly after)... Might want to resolve concerns mentioned in #2857 (comment) before releasing?

This bit:

Seems like the check [OP linked to in their comment] should be !=, not <. Otherwise changes requiring bumping installVersion have to be backwards-compatible due to the aforementioned scenario, which doesn't seem reasonable. If the installVersion do not match it seems reasonable to force a reinstall.

Would be good not to impose backward-compatibility guarantees on any stuff that depends on a particular installVersion matching, I think? And it's a ~3 character diff to fix it. I can do a PR going off of that person's comment if wanted. (Or the folks from that PR might have input if they were pinged, I guess.)


How to release / How to set the semver for the release

And (at least for stuff that happens on GitHub) the release process is basically https://github.com/google-github-actions/release-please-action#readme --> https://github.com/googleapis/release-please#readme. (And after that, I'm sure some maintainer(s) here know about how publishing the new version to the npm package registry works.)

You can manually set the version number (like 9.4.1 or 10.0.0) for release-please to follow by adding a commit with Release-As: x.x.x in the body of the commit message (below the first line)
https://github.com/googleapis/release-please#how-do-i-change-the-version-number

I did the PR to add release-please to this repo at the request of multiple active node-gyp contributors at the time, so if I can be of help by answering questions why it was set up any particular way I am happy to do so, but it was set up kind of arbitrarily just to make releases more automated than the existing process, to put it in a nutshell.

@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from 5d0471f to e62ac4fCompareOctober 27, 2023 03:43
@lukekarrys

Copy link
Copy Markdown
Member

I'm going to close this release PR so that when #2917 lands in release/v9, the new release-please action for that branch will create this v9.4.1 PR. And then I will push a new commit (as mentioned above #2878 (comment)) to create a v10.0.0 PR.

We can also use the new v10.0.0 PR to discuss any other changes that should land before the semver major is released.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@lukekarrys@legobeat@Trott@mfranzs@DeeDeeG
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

chore: release 9.4.1 - #2878

Closed
github-actions[bot] wants to merge 1 commit into
mainfrom
release-v9.4.1
Closed

chore: release 9.4.1#2878
github-actions[bot] wants to merge 1 commit into
mainfrom
release-v9.4.1

Conversation

@github-actions

@github-actionsgithub-actionsBot commented Jun 22, 2023

Copy link
Copy Markdown
Contributor

🤖 I have created a release *beep* *boop*

9.4.1 (2023-10-27)

Bug Fixes

  • create Python symlink only during builds, and clean it up after (#2721) (0f1f667)

Core

  • find python checks order changed on windows (#2872) (b030555)

Tests

Doc

  • Add note about Python symlinks (PR 2362) to CHANGELOG.md for 9.1.0 (#2783) (b3d41ae)
  • README.md Do not hardcode the supported versions of Python (#2880) (bb93b94)
  • Update windows installation instructions in README.md (#2882) (c9caa2e)

This PR was generated with Release Please. See documentation.

@lukekarrys

Copy link
Copy Markdown
Member

This shouldn't be released in it's current state. #2770 landed with a breaking change but isn't formatted like a conventional commit, so it didn't trigger this PR to be 10.0.0 as it should be.

@legobeat

legobeat commented Jul 5, 2023

Copy link
Copy Markdown

@lukekarrys Do you think we can consider #2849 for inclusion into a 9.4.1? Given that #2796 got included in 9.4.0 and is actually breaking Node 12.x support (#2873).

Then a more long-term solution which doesn't need to consider 12.x support can be targeting a 10.x release.

@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from ebae5d2 to a43758dCompareJuly 20, 2023 13:05
@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from ee68a4f to 5c042eeCompareAugust 26, 2023 08:48
@Trott

Copy link
Copy Markdown
Member

If someone does git commit --allow-empty -m "chore: release 10.0.0" -m "Release-As: 10.0.0" and pushes that to the main branch, I think Release Please will either update this PR or open a new one to release this stuff as 10.0.0. I think.....

@mfranzs

Copy link
Copy Markdown

Hi - is there any documentation on release processes here / when we can hope the latest main will be released? There are some critical fixes breaking Electron builds that we're waiting for. Thank you for this package!

@lukekarrys

Copy link
Copy Markdown
Member

There is not any current documentation @mfranzs. But my goal is to release the latest main branch this week. Although I should note that it will likely be release as a new major version 10.0.0.

@mfranzs

Copy link
Copy Markdown

Thank you very much!

@legobeat

legobeat commented Oct 3, 2023

Copy link
Copy Markdown

@lukekarrys any chance we can actually get a 9.x release? LMK if you see any blockers or other things you would need assistance with to make that happen.

^9.x is still dependended on by various unmaintained widely used ecosystem packages..

@DeeDeeG

DeeDeeG commented Oct 3, 2023

Copy link
Copy Markdown
Contributor

Stuff to consider doing before releasing?

EDIT: Besides @legobeat's point above (which I don't want to bury with my comment so shortly after)... Might want to resolve concerns mentioned in #2857 (comment) before releasing?

This bit:

Seems like the check [OP linked to in their comment] should be !=, not <. Otherwise changes requiring bumping installVersion have to be backwards-compatible due to the aforementioned scenario, which doesn't seem reasonable. If the installVersion do not match it seems reasonable to force a reinstall.

Would be good not to impose backward-compatibility guarantees on any stuff that depends on a particular installVersion matching, I think? And it's a ~3 character diff to fix it. I can do a PR going off of that person's comment if wanted. (Or the folks from that PR might have input if they were pinged, I guess.)


How to release / How to set the semver for the release

And (at least for stuff that happens on GitHub) the release process is basically https://github.com/google-github-actions/release-please-action#readme --> https://github.com/googleapis/release-please#readme. (And after that, I'm sure some maintainer(s) here know about how publishing the new version to the npm package registry works.)

You can manually set the version number (like 9.4.1 or 10.0.0) for release-please to follow by adding a commit with Release-As: x.x.x in the body of the commit message (below the first line)
https://github.com/googleapis/release-please#how-do-i-change-the-version-number

I did the PR to add release-please to this repo at the request of multiple active node-gyp contributors at the time, so if I can be of help by answering questions why it was set up any particular way I am happy to do so, but it was set up kind of arbitrarily just to make releases more automated than the existing process, to put it in a nutshell.

@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from 5d0471f to e62ac4fCompareOctober 27, 2023 03:43
@lukekarrys

Copy link
Copy Markdown
Member

I'm going to close this release PR so that when #2917 lands in release/v9, the new release-please action for that branch will create this v9.4.1 PR. And then I will push a new commit (as mentioned above #2878 (comment)) to create a v10.0.0 PR.

We can also use the new v10.0.0 PR to discuss any other changes that should land before the semver major is released.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@lukekarrys@legobeat@Trott@mfranzs@DeeDeeG
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

chore: release 9.4.1 - #2878

Closed
github-actions[bot] wants to merge 1 commit into
mainfrom
release-v9.4.1
Closed

chore: release 9.4.1#2878
github-actions[bot] wants to merge 1 commit into
mainfrom
release-v9.4.1

Conversation

@github-actions

@github-actionsgithub-actionsBot commented Jun 22, 2023

Copy link
Copy Markdown
Contributor

🤖 I have created a release *beep* *boop*

9.4.1 (2023-10-27)

Bug Fixes

  • create Python symlink only during builds, and clean it up after (#2721) (0f1f667)

Core

  • find python checks order changed on windows (#2872) (b030555)

Tests

Doc

  • Add note about Python symlinks (PR 2362) to CHANGELOG.md for 9.1.0 (#2783) (b3d41ae)
  • README.md Do not hardcode the supported versions of Python (#2880) (bb93b94)
  • Update windows installation instructions in README.md (#2882) (c9caa2e)

This PR was generated with Release Please. See documentation.

@lukekarrys

Copy link
Copy Markdown
Member

This shouldn't be released in it's current state. #2770 landed with a breaking change but isn't formatted like a conventional commit, so it didn't trigger this PR to be 10.0.0 as it should be.

@legobeat

legobeat commented Jul 5, 2023

Copy link
Copy Markdown

@lukekarrys Do you think we can consider #2849 for inclusion into a 9.4.1? Given that #2796 got included in 9.4.0 and is actually breaking Node 12.x support (#2873).

Then a more long-term solution which doesn't need to consider 12.x support can be targeting a 10.x release.

@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from ebae5d2 to a43758dCompareJuly 20, 2023 13:05
@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from ee68a4f to 5c042eeCompareAugust 26, 2023 08:48
@Trott

Copy link
Copy Markdown
Member

If someone does git commit --allow-empty -m "chore: release 10.0.0" -m "Release-As: 10.0.0" and pushes that to the main branch, I think Release Please will either update this PR or open a new one to release this stuff as 10.0.0. I think.....

@mfranzs

Copy link
Copy Markdown

Hi - is there any documentation on release processes here / when we can hope the latest main will be released? There are some critical fixes breaking Electron builds that we're waiting for. Thank you for this package!

@lukekarrys

Copy link
Copy Markdown
Member

There is not any current documentation @mfranzs. But my goal is to release the latest main branch this week. Although I should note that it will likely be release as a new major version 10.0.0.

@mfranzs

Copy link
Copy Markdown

Thank you very much!

@legobeat

legobeat commented Oct 3, 2023

Copy link
Copy Markdown

@lukekarrys any chance we can actually get a 9.x release? LMK if you see any blockers or other things you would need assistance with to make that happen.

^9.x is still dependended on by various unmaintained widely used ecosystem packages..

@DeeDeeG

DeeDeeG commented Oct 3, 2023

Copy link
Copy Markdown
Contributor

Stuff to consider doing before releasing?

EDIT: Besides @legobeat's point above (which I don't want to bury with my comment so shortly after)... Might want to resolve concerns mentioned in #2857 (comment) before releasing?

This bit:

Seems like the check [OP linked to in their comment] should be !=, not <. Otherwise changes requiring bumping installVersion have to be backwards-compatible due to the aforementioned scenario, which doesn't seem reasonable. If the installVersion do not match it seems reasonable to force a reinstall.

Would be good not to impose backward-compatibility guarantees on any stuff that depends on a particular installVersion matching, I think? And it's a ~3 character diff to fix it. I can do a PR going off of that person's comment if wanted. (Or the folks from that PR might have input if they were pinged, I guess.)


How to release / How to set the semver for the release

And (at least for stuff that happens on GitHub) the release process is basically https://github.com/google-github-actions/release-please-action#readme --> https://github.com/googleapis/release-please#readme. (And after that, I'm sure some maintainer(s) here know about how publishing the new version to the npm package registry works.)

You can manually set the version number (like 9.4.1 or 10.0.0) for release-please to follow by adding a commit with Release-As: x.x.x in the body of the commit message (below the first line)
https://github.com/googleapis/release-please#how-do-i-change-the-version-number

I did the PR to add release-please to this repo at the request of multiple active node-gyp contributors at the time, so if I can be of help by answering questions why it was set up any particular way I am happy to do so, but it was set up kind of arbitrarily just to make releases more automated than the existing process, to put it in a nutshell.

@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from 5d0471f to e62ac4fCompareOctober 27, 2023 03:43
@lukekarrys

Copy link
Copy Markdown
Member

I'm going to close this release PR so that when #2917 lands in release/v9, the new release-please action for that branch will create this v9.4.1 PR. And then I will push a new commit (as mentioned above #2878 (comment)) to create a v10.0.0 PR.

We can also use the new v10.0.0 PR to discuss any other changes that should land before the semver major is released.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@lukekarrys@legobeat@Trott@mfranzs@DeeDeeG
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content

chore: release 9.4.1 - #2878

Closed
github-actions[bot] wants to merge 1 commit into
mainfrom
release-v9.4.1
Closed

chore: release 9.4.1#2878
github-actions[bot] wants to merge 1 commit into
mainfrom
release-v9.4.1

Conversation

@github-actions

@github-actionsgithub-actionsBot commented Jun 22, 2023

Copy link
Copy Markdown
Contributor

🤖 I have created a release *beep* *boop*

9.4.1 (2023-10-27)

Bug Fixes

  • create Python symlink only during builds, and clean it up after (#2721) (0f1f667)

Core

  • find python checks order changed on windows (#2872) (b030555)

Tests

Doc

  • Add note about Python symlinks (PR 2362) to CHANGELOG.md for 9.1.0 (#2783) (b3d41ae)
  • README.md Do not hardcode the supported versions of Python (#2880) (bb93b94)
  • Update windows installation instructions in README.md (#2882) (c9caa2e)

This PR was generated with Release Please. See documentation.

@lukekarrys

Copy link
Copy Markdown
Member

This shouldn't be released in it's current state. #2770 landed with a breaking change but isn't formatted like a conventional commit, so it didn't trigger this PR to be 10.0.0 as it should be.

@legobeat

legobeat commented Jul 5, 2023

Copy link
Copy Markdown

@lukekarrys Do you think we can consider #2849 for inclusion into a 9.4.1? Given that #2796 got included in 9.4.0 and is actually breaking Node 12.x support (#2873).

Then a more long-term solution which doesn't need to consider 12.x support can be targeting a 10.x release.

@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from ebae5d2 to a43758dCompareJuly 20, 2023 13:05
@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from ee68a4f to 5c042eeCompareAugust 26, 2023 08:48
@Trott

Copy link
Copy Markdown
Member

If someone does git commit --allow-empty -m "chore: release 10.0.0" -m "Release-As: 10.0.0" and pushes that to the main branch, I think Release Please will either update this PR or open a new one to release this stuff as 10.0.0. I think.....

@mfranzs

Copy link
Copy Markdown

Hi - is there any documentation on release processes here / when we can hope the latest main will be released? There are some critical fixes breaking Electron builds that we're waiting for. Thank you for this package!

@lukekarrys

Copy link
Copy Markdown
Member

There is not any current documentation @mfranzs. But my goal is to release the latest main branch this week. Although I should note that it will likely be release as a new major version 10.0.0.

@mfranzs

Copy link
Copy Markdown

Thank you very much!

@legobeat

legobeat commented Oct 3, 2023

Copy link
Copy Markdown

@lukekarrys any chance we can actually get a 9.x release? LMK if you see any blockers or other things you would need assistance with to make that happen.

^9.x is still dependended on by various unmaintained widely used ecosystem packages..

@DeeDeeG

DeeDeeG commented Oct 3, 2023

Copy link
Copy Markdown
Contributor

Stuff to consider doing before releasing?

EDIT: Besides @legobeat's point above (which I don't want to bury with my comment so shortly after)... Might want to resolve concerns mentioned in #2857 (comment) before releasing?

This bit:

Seems like the check [OP linked to in their comment] should be !=, not <. Otherwise changes requiring bumping installVersion have to be backwards-compatible due to the aforementioned scenario, which doesn't seem reasonable. If the installVersion do not match it seems reasonable to force a reinstall.

Would be good not to impose backward-compatibility guarantees on any stuff that depends on a particular installVersion matching, I think? And it's a ~3 character diff to fix it. I can do a PR going off of that person's comment if wanted. (Or the folks from that PR might have input if they were pinged, I guess.)


How to release / How to set the semver for the release

And (at least for stuff that happens on GitHub) the release process is basically https://github.com/google-github-actions/release-please-action#readme --> https://github.com/googleapis/release-please#readme. (And after that, I'm sure some maintainer(s) here know about how publishing the new version to the npm package registry works.)

You can manually set the version number (like 9.4.1 or 10.0.0) for release-please to follow by adding a commit with Release-As: x.x.x in the body of the commit message (below the first line)
https://github.com/googleapis/release-please#how-do-i-change-the-version-number

I did the PR to add release-please to this repo at the request of multiple active node-gyp contributors at the time, so if I can be of help by answering questions why it was set up any particular way I am happy to do so, but it was set up kind of arbitrarily just to make releases more automated than the existing process, to put it in a nutshell.

@github-actions
github-actionsBotforce-pushed the release-v9.4.1 branch 2 times, most recently from 5d0471f to e62ac4fCompareOctober 27, 2023 03:43
@lukekarrys

Copy link
Copy Markdown
Member

I'm going to close this release PR so that when #2917 lands in release/v9, the new release-please action for that branch will create this v9.4.1 PR. And then I will push a new commit (as mentioned above #2878 (comment)) to create a v10.0.0 PR.

We can also use the new v10.0.0 PR to discuss any other changes that should land before the semver major is released.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@lukekarrys@legobeat@Trott@mfranzs@DeeDeeG