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

Refactor GSoc General Guidelines #45

Merged
merged 2 commits into from
Feb 16, 2024
Merged

Conversation

Dilini-457
Copy link
Contributor

@Dilini-457 Dilini-457 commented Feb 4, 2024

To suggest a change to the Handbook, please review the Handbook Guidlines section.

Purpose:

The purpose of this PR is to fix #44

Related to:

Please provide a brief explanation for this change:

Preview Link:

https://pr-45-sef-handbook.surge.sh/

Please indicate the types of revisions being suggested for the Handbook (please check all that apply):

  • Small improvement (typos, clarifications, etc.)
  • Adding a new section
  • [ x ] Modifying existing section
  • Documenting a new process
  • Adding a new page or directory
  • Other

Author Checklist

  • [ x ] Provided a concise title for the PR
  • Provided a brief explanation for this change (Say why not just what)
    • (Attach screenshots, Slack conversations, etc. as necessary)
  • Indicated the types of changes included in this PR
  • Verified that no confidential data is in this PR
  • Posted an update in #handbook channel
  • If the changes affect team members, or warrant an announcement in another way, please consider posting an update in #general channel

Copy link
Member

@anjula-sack anjula-sack left a comment

Choose a reason for hiding this comment

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

Requested some minor changes @Dilini-457

docs/gsoc-contributor-guidance.md Outdated Show resolved Hide resolved
docs/gsoc-contributor-guidance.md Outdated Show resolved Hide resolved
docs/gsoc-contributor-guidance.md Outdated Show resolved Hide resolved
@anjula-sack
Copy link
Member

@Dilini-457 can you update the PR with the changes?

1 similar comment
@anjula-sack
Copy link
Member

@Dilini-457 can you update the PR with the changes?

@Dilini-457
Copy link
Contributor Author

@anjula-sack Updated the PR with requested changes.

Copy link
Member

@anjula-sack anjula-sack left a comment

Choose a reason for hiding this comment

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

Thank you! @Dilini-457

@anjula-sack anjula-sack merged commit 7355b12 into sef-global:main Feb 16, 2024
1 check passed
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.

Rephrase the contributors guidelines
2 participants