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.

Sign up for GitHub

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

Jump to bottom

summaries and intros - getting started section #504

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

summaries and intros - getting started section #504

wants to merge 9 commits into from

Conversation

iamdamion
Copy link
Contributor

Finished up all to do items on the 3rd eval milestone.

This closes #494

@iamdamion @bendhouseart
Update docs/getting_started/folders_and_files/index.md
0d87c64 
looks good

Co-authored-by: Anthony Galassi <28850131+bendhouseart@users.noreply.github.com>
Remi-Gau
Remi-Gau requested changes Sep 26, 2024
View reviewed changes
Copy link
Contributor

@Remi-Gau Remi-Gau left a comment

Choose a reason for hiding this comment

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

Can you use semantic linebreaks ?

https://sembr.org/

iamdamion and others added 2 commits September 26, 2024 09:36
@iamdamion @Remi-Gau
Update docs/getting_started/folders_and_files/index.md
6aa8d92 
thanks

Co-authored-by: Remi Gau <remi_gau@hotmail.com>
@iamdamion
Update index.md
f84115a 
added breaks and updated text.
christinerogers
christinerogers requested changes Sep 26, 2024
View reviewed changes
Copy link
Contributor

@christinerogers christinerogers left a comment

Choose a reason for hiding this comment

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

  • please see inline comments
  • please see text change suggestions inline,
  • give it another read-over out loud, and :
  • add a note at the top of Getting Started to say this is a continuation of the BIDS Starter Kit project.

docs/getting_started/folders_and_files/index.md Outdated Show resolved Hide resolved
docs/getting_started/resources/talks.md Outdated Show resolved Hide resolved
docs/getting_started/tutorials/index.md Outdated Show resolved Hide resolved
docs/getting_started/resources/index.md Outdated Show resolved Hide resolved
docs/getting_started/tutorials/conversion/index.md Outdated Show resolved Hide resolved
iamdamion and others added 5 commits September 26, 2024 09:52
@iamdamion
Update talks.md
5c76c62 
break added
@iamdamion @christinerogers
Update docs/getting_started/tutorials/index.md
7aaf566 
added breaks, cleaned up wording, added issue link for submitting new tutorials.

Co-authored-by: christinerogers <christinerogers@users.noreply.github.com>
@iamdamion @christinerogers
Update docs/getting_started/tutorials/conversion/index.md
6ed4a36 
simplified and edited per suggestions

Co-authored-by: christinerogers <christinerogers@users.noreply.github.com>
@iamdamion
Update index.md
878b962 
updated per suggestions
@iamdamion
Update index.md
a68e4be 
small wording edit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Remi-Gau Remi-Gau Remi-Gau requested changes

@bendhouseart bendhouseart bendhouseart left review comments

@christinerogers christinerogers christinerogers requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@iamdamion @christinerogers @Remi-Gau @bendhouseart