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

Data Science with Cybersecurity #216

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

vrm-piyush
Copy link

A blog based on utilising data science in the field of cybersecurity.

@vrm-piyush vrm-piyush marked this pull request as ready for review September 20, 2021 17:23
@vrm-piyush vrm-piyush closed this Sep 20, 2021
@vrm-piyush vrm-piyush reopened this Sep 20, 2021
Gemfile.lock Outdated
@@ -249,6 +249,7 @@ GEM
tzinfo (1.2.5)
thread_safe (~> 0.1)
unicode-display_width (1.5.0)
wdm (0.1.1)
Copy link
Member

Choose a reason for hiding this comment

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

please dont change this file

Copy link
Author

Choose a reason for hiding this comment

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

Ok
Why did the change happen in my file?

Copy link
Member

Choose a reason for hiding this comment

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

because your local environment might have changed these files,
You should avoid overwriting these files because they may cause trouble in deployment.

As a sanity check, we review the PR and look for such changes, so code doesn't break when pushed to prod.

Copy link
Member

@AvikantSrivastava AvikantSrivastava left a comment

Choose a reason for hiding this comment

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

The changes look good to me, can you confirm if the site was running fine on local?

@AvikantSrivastava
Copy link
Member

@vrm-piyush check if these routes work fine on your local
/author-piyush-verma.html
/Data-Science-with-Cybersecurity

@vrm-piyush
Copy link
Author

@vrm-piyush check if these routes work fine on your local /author-piyush-verma.html /Data-Science-with-Cybersecurity

Yeah, its working fine.

@AvikantSrivastava
Copy link
Member

Reset contents ofGemfile.lock to master, and I'll help you merge your PR

@vrm-piyush
Copy link
Author

vrm-piyush commented Oct 4, 2021 via email

@AvikantSrivastava
Copy link
Member

@vrm-piyush
Copy link
Author

vrm-piyush commented Oct 5, 2021 via email

@AvikantSrivastava
Copy link
Member

@vrm-piyush Just undo the changes in Gemfile.lock file and make a commit

@vrm-piyush
Copy link
Author

vrm-piyush commented Oct 7, 2021 via email

@vercel
Copy link

vercel bot commented Oct 16, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/glugmvit/glugmvit-github-io/Ft4BboHk1VWJnd8Hy99mvZ6Pk9VX
✅ Preview: https://glugmvit-github-io-git-fork-vrm-piyush-master-glugmvit.vercel.app

@vrm-piyush
Copy link
Author

Hey! The blog hasn't been uploaded yet. Is there any issue?

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

Successfully merging this pull request may close these issues.

None yet

2 participants