Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Check out PR branch for explorer CI checks #20750

Merged
merged 1 commit into from
Oct 19, 2021
Merged

Conversation

carllin
Copy link
Contributor

@carllin carllin commented Oct 17, 2021

Problem

Changes to explorer currently always run against the master branch

Summary of Changes

Fixes #

@carllin carllin force-pushed the TestW branch 2 times, most recently from ef358f8 to 2dc32a5 Compare October 17, 2021 09:04
@carllin
Copy link
Contributor Author

carllin commented Oct 17, 2021

I merged this change into my own repository's master branch: carllin@2dc32a5#diff-b1fc48c49c55190ca785814f56c784011e147d731250768cd38d893e312ba02dR22-R23, submitted a PR with the MetaplexNFTArt.tsx formatting changes here: carllin#289, and verified it passes the formatting checks
https://github.com/carllin/solana/runs/3918047111?check_suite_focus=true.

@carllin carllin requested review from jstarry and oJshua October 17, 2021 09:19
@carllin carllin changed the title test Check out PR branch for explorer CI checks Oct 17, 2021
@jstarry
Copy link
Contributor

jstarry commented Oct 17, 2021

I think this commit might have changed which commit cha is checked out: 49af534 @axleiro can you take a look?

@jstarry jstarry requested a review from denispalab October 17, 2021 19:20
@jstarry jstarry merged commit 27d1850 into solana-labs:master Oct 19, 2021
@jstarry
Copy link
Contributor

jstarry commented Oct 19, 2021

Merging for now to fix CI issues until @axleiro gets a chance to take a look

dankelleher pushed a commit to identity-com/solana that referenced this pull request Nov 24, 2021
frits-metalogix added a commit to identity-com/solana that referenced this pull request Nov 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants