Skip to content

Conversation

@krauspt
Copy link
Contributor

@krauspt krauspt commented Dec 18, 2023

Issue Addressed

I didn't create a specific issue, because changes are trivial, and only documentation is updated.

Proposed Changes

I've found some outdated links to external resources, this PR fixes them.

Additional Info

No additional info.

@CLAassistant
Copy link

CLAassistant commented Dec 18, 2023

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@chong-he chong-he left a comment

Choose a reason for hiding this comment

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

Thanks for the fix, looks good to me

The [formula][] is kept up-to-date by the Homebrew community and a bot that lists for new releases.

The package source can be found in the [homebrew-core](https://github.com/Homebrew/homebrew-core/blob/master/Formula/lighthouse.rb) repo.
The package source can be found in the [homebrew-core](https://github.com/Homebrew/homebrew-core/blob/master/Formula/l/lighthouse.rb) repo.
Copy link
Member

Choose a reason for hiding this comment

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

This is fixed in #4822 in the unstable branch but I see no conflict so it should be good

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep, my mistake I should start from the unstable branch instead of stable ...

@chong-he chong-he added docs Documentation ready-for-review The code is ready for review labels Dec 19, 2023
@krauspt
Copy link
Contributor Author

krauspt commented Jan 7, 2024

@chong-he is there anything from my side that needs to be done?

@chong-he
Copy link
Member

chong-he commented Jan 7, 2024

@chong-he is there anything from my side that needs to be done?

No, this is ready to be merged once it passes all checks, thanks!

@chong-he chong-he removed the ready-for-review The code is ready for review label Jan 7, 2024
@dapplion dapplion added the ready-for-merge This PR is ready to merge. label Jan 29, 2024
@michaelsproul
Copy link
Member

@Mergifyio queue

@mergify
Copy link

mergify bot commented Jan 29, 2024

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at 47f05ac

mergify bot added a commit that referenced this pull request Jan 29, 2024
@mergify mergify bot merged commit 47f05ac into sigp:unstable Jan 30, 2024
danielrachi1 pushed a commit to danielrachi1/lighthouse that referenced this pull request Feb 14, 2024
* fix: update outdated links to external resources

* Update style guide link
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation ready-for-merge This PR is ready to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants