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

misc: rename lantern master values to baseline values #10950

Merged
merged 5 commits into from
Feb 24, 2021

Conversation

connorjclark
Copy link
Collaborator

No description provided.

@connorjclark connorjclark requested a review from a team as a code owner June 12, 2020 07:52
@connorjclark connorjclark requested review from patrickhulce and removed request for a team June 12, 2020 07:52
Copy link
Collaborator

@patrickhulce patrickhulce left a comment

Choose a reason for hiding this comment

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

FWIW, these are named master for the fact that they should be inline with whatever is true on master branch and not some immutable baseline.

I'd have a strong preference for renaming these lantern-trunk-* and renaming our master branch to trunk instead. Do we think that's feasible to do quickly? (If not I'm fine with baseline in the meantime)

Also missing a few other master usages in typedefs and filenames :)
(flagged by tests)

@brendankenny
Copy link
Member

I'd have a strong preference for renaming these lantern-trunk-* and renaming our master branch to trunk instead. Do we think that's feasible to do quickly? (If not I'm fine with baseline in the meantime)

you've convinced me on trunk over main. I'm also fine to call these lantern-trunk-* even while we figure out the logistics of using trunk for the default branch.

It would be nice if we all agreed on definitive meanings for trunk/baseline/golden(/others?), though :) It sounds like you might already have a classification you're working off of @patrickhulce?

They all could mean slightly different things, but I think in practice we use them in overlapping ways.

@patrickhulce
Copy link
Collaborator

It would be nice if we all agreed on definitive meanings for trunk/baseline/golden(/others?), though :) It sounds like you might already have a classification you're working off of @patrickhulce?

I'm defensive of losing the signal in these names because this was the result of me and Paul (I thought it was you too but if you're asking then I guess not?) sitting in a room for like an hour trying to work out the clearest way to communicate what these are after my attempt didn't cut it 😆

Preemptively renaming to trunk ahead of branch rename WFM 👍

@connorjclark
Copy link
Collaborator Author

connorjclark commented Jun 12, 2020

I was going to recommend "main" for the branch rename, primarily because: that's what Chromium is leaning towards (at this early stage); and "maTAB" autocomplete would be maintained for muscle memory. if everyone did "main" you'd never have to recall whatever name the project you're working on chose.

@brendankenny
Copy link
Member

I'm defensive of losing the signal in these names because this was the result of me and Paul sitting in a room for like an hour trying to work out the clearest way to communicate what these are after my attempt didn't cut it 😆

I'm good with it (and whatever, really), I just want it written down in an issue or a comment or somewhere so the next time I'm naming something and I'm like "wait, should this be a golden-lhr or a trunk-lhr?" I can just look at that and not have to think any more than that about it :D

(I thought it was you too but if you're asking then I guess not?)

ha, well that's very possible

@patrickhulce
Copy link
Collaborator

@connorjclark I believe our discussed settled on we will hold off renaming master until full consensus has been reached in other places for the name we're going to match. Are you OK with tabling this for now to avoid a temporary inconsistent name?

AFAIK, no one other than you and me has had to even notice these files exist in the course of their Lighthouse work, so I'm less concerned about the impact here compared to the branch itself (which we're already OK temporarily postpoing).

@patrickhulce
Copy link
Collaborator

Given that the community has settled on main and lantern-main would be even more ambiguous (unfortunately confirming my issues with main as opposed to something like trunk 😢 ), I'm fine moving forward with baseline here if everyone still thinks it's clear what it's referring to.

Copy link
Collaborator

@patrickhulce patrickhulce left a comment

Choose a reason for hiding this comment

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

looks like there's more to update, but ya I'm fine with the master -> baseline rename :)

Sign up for free to 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.

6 participants