Skip to content

Onboarding issues I noticed #105211

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

jdmarshall
Copy link
Contributor

What is this feature?

Formatting difference between sections makes 'make run' a bit harder to pick out than the rest of the instructions.

And I'm fairly certain the filecount has gotten big enough that ulimit -n 4096 doesn't cut the mustard.

Why do we need this feature?

Make it a little easier for new contributors to come onboard.

Who is this feature for?

first time developers

Which issue(s) does this PR fix?:

No issue

Special notes for your reviewer:

Please check that:

  • [x ] It works as expected from a user's perspective.
  • If this is a pre-GA feature, it is behind a feature toggle.
  • The docs are updated, and if this is a notable improvement, it's added to our What's New doc.

On a bare system I'm getting almost 6200 files from the given `find` command.
It's clear that this project has grown considerably since this line was added
to the onboarding docs.
@jdmarshall jdmarshall requested a review from a team as a code owner May 10, 2025 23:43
@jdmarshall jdmarshall requested review from moxious and usmangt and removed request for a team May 10, 2025 23:43
@CLAassistant
Copy link

CLAassistant commented May 10, 2025

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added this to the 12.1.x milestone May 10, 2025
@jdmarshall jdmarshall requested a review from a team as a code owner May 20, 2025 02:26
@github-actions github-actions bot added the type/docs Flags the technical writing team for documentation support; auto adds to org-wide docs project label May 20, 2025
@irenerl24 irenerl24 added the no-changelog Skip including change in changelog/release notes label May 20, 2025
@irenerl24
Copy link
Collaborator

hi @jdmarshall! could you please sign the CLA so that we can go ahead and merge the PR? Thank you in advance! :)

@jdmarshall
Copy link
Contributor Author

Done

Copy link
Contributor

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 2 weeks if no further activity occurs. Please feel free to give a status update or ping for review. Thank you for your contributions!

@github-actions github-actions bot added the stale Issue with no recent activity label Jun 20, 2025
@jdmarshall
Copy link
Contributor Author

still would be good to merge

@github-actions github-actions bot removed the stale Issue with no recent activity label Jun 21, 2025
@tonypowa tonypowa added the no-backport Skip backport of PR label Jun 24, 2025
@jdmarshall
Copy link
Contributor Author

What's up with:

Error: Error message: Unable to get ACTIONS_ID_TOKEN_REQUEST_URL env variable

Do I need to rebase?

Copy link
Contributor

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 2 weeks if no further activity occurs. Please feel free to give a status update or ping for review. Thank you for your contributions!

@github-actions github-actions bot added the stale Issue with no recent activity label Jul 25, 2025
@jdmarshall
Copy link
Contributor Author

ping

@github-actions github-actions bot removed the stale Issue with no recent activity label Jul 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-backport Skip backport of PR no-changelog Skip including change in changelog/release notes type/docs Flags the technical writing team for documentation support; auto adds to org-wide docs project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants