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

docs: Update that the Tour De Sun 22 is now closed #1755

Merged
merged 10 commits into from
Jun 30, 2024

Conversation

web3validator
Copy link

Problem

The current documentation includes outdated information about the Tour De Sun 22 program, which is now closed to new applicants. This could cause confusion for users who are trying to participate in the program.

Summary of Changes

  • Updated the Tour De Sun 22 section to indicate that the program is winding down and applications are now closed.
  • Added information that March 2024 was the final month of the program.
  • Included a link to the official blog post announcing the end of Tour De Sun 22 for further details.

Fixes #

- Updated the Tour De Sun 22 section to indicate that the program is winding down and applications are now closed.
- Added information that March 2024 was the final month of the program.
- Included a link to the official blog post announcing the end of Tour De Sun 22 for further details.
@mergify mergify bot requested a review from a team June 16, 2024 05:31
The Solana ecosystem operates various stake pools. These stake pools delegate stake to validators using various delegation strategies. To learn more about stake pools and their delegation strategies, visit the [Solana Foundation stake pools page](https://solana.org/stake-pools).
Copy link

Choose a reason for hiding this comment

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

This chunk shouldn't be in the diff - I don't see any changes to the actual content so perhaps a line ending issue ?

Copy link

Choose a reason for hiding this comment

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

This item is not resolved - there are no changes to this text so this line should not be in the diff

Copy link
Author

Choose a reason for hiding this comment

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

Yes, it is the problem with editor

Copy link
Author

Choose a reason for hiding this comment

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

I fixed it, @steviez check it.

docs/src/operations/validator-initiatives.md Outdated Show resolved Hide resolved
@steviez
Copy link

steviez commented Jun 28, 2024

Just incase - I see you merged a commit, but I don't see any actual differences / the commit you pushed is empty. And by the way, it could be a line ending issue for why the extra lines are showing up at the end of the diff. Are you editing on a Windows machine ?

shortened the information regarding TDS-22
@web3validator
Copy link
Author

I have edited the PR. Kindly check if it looks correct now.

Copy link

@steviez steviez left a comment

Choose a reason for hiding this comment

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

I have edited the PR. Kindly check if it looks correct now.

Thanks for the continued additions. The first segment needs one more word (this was a mistake on my part, apologies). However, the last line of the file is still showing as changed but should NOT be:
image

As I mentioned, this might be an issue with your editor / system using different line endings

docs/src/operations/validator-initiatives.md Outdated Show resolved Hide resolved
Copy link

@steviez steviez left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks!

@steviez steviez changed the title Update Tour De Sun 22 program information docs: Update that the Tour De Sun 22 is now closed Jun 30, 2024
@steviez steviez merged commit bd1a5df into anza-xyz:master Jun 30, 2024
1 of 2 checks passed
samkim-crypto pushed a commit to samkim-crypto/agave that referenced this pull request Jul 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants