-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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 security-getting-started-tutorial from QE feedback #45686
Conversation
Thanks for your pull request! Your pull request does not follow our editorial rules. Could you have a look?
This message is automatically generated by a bot. |
I'm not sure we need to add file path locations, users do no create those files manually, making sure the directory structure matches the package name, they put it into IDE and IDE creates everything for them. |
🙈 The PR is closed and the preview is expired. |
c345864
to
8d3e117
Compare
Thanks, Sergey! While IDEs handle directory structures for most users, including file paths in code block labels adds clarity, reassures alignment with expectations, and helps those troubleshooting or using less automated setups. I appreciate the discussion! |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@rolfedh Hi, looks like you need to recreate this PR since it contains a merge commit |
81c2807
to
b15f1ad
Compare
Sorry about that. Rebased and -f pushed with just one commit now. Thanks. |
Status for workflow
|
Update security-getting-started-tutorial from QE feedback.