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.

Sign up for GitHub

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

Jump to bottom

[action] First pass at common CI #2

Merged
merged 1 commit into from
Oct 10, 2022
Merged

[action] First pass at common CI #2

merged 1 commit into from
Oct 10, 2022

Conversation

dopey
Copy link

@dopey dopey commented Oct 8, 2022

No description provided.

@dopey dopey requested a review from maraino October 8, 2022 23:05
@dopey dopey self-assigned this Oct 8, 2022
maraino
maraino approved these changes Oct 10, 2022
View reviewed changes
Copy link

@maraino maraino left a comment

Choose a reason for hiding this comment

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

Aren't we suppose to use Mozilla's now that it is up-to-date?

@dopey
Copy link
Author

dopey commented Oct 10, 2022

Aren't we suppose to use Mozilla's now that it is up-to-date?

Via Herman -- ```
we can’t use the original. The replace is pointing to a specific branch with the patches applied. There has been no movement in the original since end of last year; same status as when I created the fork.

@dopey dopey merged commit 5660c44 into master Oct 10, 2022
@dopey dopey deleted the max/ci branch October 10, 2022 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maraino maraino maraino approved these changes

Assignees

@dopey dopey

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dopey @maraino