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

Bump recorder version to v1.5.0 #537

Open
wants to merge 128 commits into
base: main
Choose a base branch
from
Open

Conversation

justinkambic
Copy link
Contributor

Bumps the recorder version to 1.5.0. The old README specifies the dev should simply push this to main, but we no longer have permission to do that so a PR is the route it must take.

@@ -309,218 +309,6 @@ WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
OTHER DEALINGS IN THE SOFTWARE.

---
This product relies on playwright-core
Copy link
Member

Choose a reason for hiding this comment

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

Oops again? Something wrong with our tooling?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ths has consistently failed for a really long time, needs fixed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Playwright core does not include a license file in the package, thus when it gets installed there's no license for the script to pick up and include in NOTICE.txt.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hey - I added some automation to pull down the correct license file for the current tag in node_modules - see here: 6dd49b0

This should fix the playwright bit that always gets deleted or needs manual inclusion.

justinkambic and others added 4 commits January 13, 2025 13:08
order matters

Update .buildkite/release.yml

Update .buildkite/release.yml

help with the name of the step

docs and debug
@@ -36,17 +37,24 @@ steps:

- wait: ~

- label: Download signed artifacts
# The order matters here, we need to download the gpg files first, then the windows and macos files
Copy link
Member

Choose a reason for hiding this comment

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

See #541

.buildkite/scripts/package.sh Outdated Show resolved Hide resolved
@justinkambic
Copy link
Contributor Author

@v1v can you have a look at the packaging build steps here: 8648d56

My intent is to have all the steps run separately in parallel; it seemed to work on my most recent run.

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.

3 participants