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

Update index.md #1120

Closed
wants to merge 1 commit into from
Closed

Update index.md #1120

wants to merge 1 commit into from

Conversation

ijunxyz123
Copy link

slsa.dev ALIAS apex-loadbalancer.netlify.com
www.slsa.dev CNAME slsa.netlify.appslsa.devwww.slsa.devapex-loadbalancer.netlify.com../tools/diff_site deploy-latest.zip deploy-preview.zipslsa.netlify.appdeploy-preview.zipdeploy-latest.zipnetlify dev -c 'bundle exec jekyll serve --livereload --incremental'# Prepare version A
$ bundle exec jekyll build
$ mv _site _site.A

Prepare version B

$ bundle exec jekyll build
$ mv _site _site.B

Run the script

$ ../tools/diff_site _site.A _site.Bnetlify dev

Signed-off-by: ijunxyz123 <ijunxyz123@gmail.com>
Copy link

netlify bot commented Aug 24, 2024

Deploy Preview for slsa ready!

Name Link
🔨 Latest commit be87b4d
🔍 Latest deploy log https://app.netlify.com/sites/slsa/deploys/66c9ac7237aabc000815d205
😎 Deploy Preview https://deploy-preview-1120--slsa.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Author

@ijunxyz123 ijunxyz123 left a comment

Choose a reason for hiding this comment

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

ijunxyz123:patch-1

Copy link

@mihaimaruseac mihaimaruseac left a comment

Choose a reason for hiding this comment

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

I don't think this is a correct change. You are just making the YAML frontmatter no longer work.

@marcelamelara
Copy link
Contributor

This looks like a reputation farming attempt. I suggest we close this unless the author can provide reasoning for this change.

@lehors
Copy link
Member

lehors commented Aug 27, 2024

This PR has no value.

@lehors lehors closed this Aug 27, 2024
@albertanastasia
Copy link

albertanastasia:patch-1LICENSE.md

Copy link

@albertanastasia albertanastasia left a comment

Choose a reason for hiding this comment

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

albertanastasia:patch-1LICENSE.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants