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

Update documentation (Fall 2023) #865

Merged
merged 14 commits into from
Sep 7, 2023
Merged

Update documentation (Fall 2023) #865

merged 14 commits into from
Sep 7, 2023

Conversation

EthanThatOneKid
Copy link
Owner

Resolves #864.

@netlify
Copy link

netlify bot commented May 25, 2023 •
edited
Loading

✅ Deploy Preview for acmcsuf ready!

Name Link
🔨 Latest commit 82d1a77
🔍 Latest deploy log https://app.netlify.com/sites/acmcsuf/deploys/64f957be2ae6b60008aaf144
😎 Deploy Preview https://deploy-preview-865--acmcsuf.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@EthanThatOneKid EthanThatOneKid changed the title Update documentation (Spring semester 2023) Update documentation (Fall 2023) Sep 5, 2023
EthanThatOneKid
EthanThatOneKid commented Sep 6, 2023
View reviewed changes
Copy link
Owner Author

@EthanThatOneKid EthanThatOneKid left a comment

Choose a reason for hiding this comment

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

Should be good to go. This PR has been sitting for so long that I lost track of why this PR was not merged.

@EthanThatOneKid EthanThatOneKid marked this pull request as ready for review September 6, 2023 03:26
@EthanThatOneKid EthanThatOneKid enabled auto-merge (squash) September 6, 2023 03:26
karnikaavelumani
karnikaavelumani approved these changes Sep 7, 2023
View reviewed changes
Copy link
Collaborator

@karnikaavelumani karnikaavelumani left a comment

Choose a reason for hiding this comment

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

A couple nits but overall it looks great!

GETTING_STARTED.md Outdated Show resolved Hide resolved
GETTING_STARTED.md Outdated Show resolved Hide resolved
@EthanThatOneKid EthanThatOneKid merged commit ee5a645 into main Sep 7, 2023
10 checks passed
karnikaavelumani
karnikaavelumani reviewed Sep 7, 2023
View reviewed changes
Copy link
Collaborator

@karnikaavelumani karnikaavelumani left a comment

Choose a reason for hiding this comment

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

Nice 😎

EthanThatOneKid added a commit that referenced this pull request Sep 7, 2023
@EthanThatOneKid
Squashed commit of the following:
b857a01 
commit 77d300b
Merge: bc31a44 ee5a645
Author: EthanThatOneKid <31261035+EthanThatOneKid@users.noreply.github.com>
Date:   Wed Sep 6 22:20:11 2023 -0700

    Update CONTRIBUTING.md

commit bc31a44
Author: EthanThatOneKid <31261035+EthanThatOneKid@users.noreply.github.com>
Date:   Wed Sep 6 22:19:23 2023 -0700

    Run `npm run all`

commit 6f8e765
Author: EthanThatOneKid <31261035+EthanThatOneKid@users.noreply.github.com>
Date:   Wed Sep 6 22:02:56 2023 -0700

    CONTRIBUTING.md: add shortlinks and blog posts

commit 82d1a77
Author: EthanThatOneKid <31261035+EthanThatOneKid@users.noreply.github.com>
Date:   Wed Sep 6 21:50:45 2023 -0700

    Getting started: clarify review cycle

    Resolves <#865 (comment)>.

commit 9a0a50f
Author: EthanThatOneKid <31261035+EthanThatOneKid@users.noreply.github.com>
Date:   Wed Sep 6 21:36:09 2023 -0700

    Fix "Getting started" title

commit 24e54d0
Author: EthanThatOneKid <31261035+EthanThatOneKid@users.noreply.github.com>
Date:   Wed Sep 6 21:35:49 2023 -0700

    Include periods on complete sentences

    Resolves <#865 (comment)>.

commit 626bd00
Merge: ead0144 f376ccc
Author: EthanThatOneKid <31261035+EthanThatOneKid@users.noreply.github.com>
Date:   Wed Sep 6 21:33:33 2023 -0700

    Merge branch 'fix/864' of https://github.com/EthanThatOneKid/acmcsuf.com into fix/864

commit ead0144
Merge: 71166d0 b8de38e
Author: EthanThatOneKid <31261035+EthanThatOneKid@users.noreply.github.com>
Date:   Wed Sep 6 21:33:18 2023 -0700

    Merge branch 'main' into fix/864

commit f376ccc
Merge: 00754b4 dc4cb81
Author: Ethan Davidson <31261035+EthanThatOneKid@users.noreply.github.com>
Date:   Wed Sep 6 21:21:24 2023 -0700

    Merge branch 'main' into fix/864

commit 00754b4
Merge: 71166d0 5a7db0c
Author: Ethan Davidson <31261035+EthanThatOneKid@users.noreply.github.com>
Date:   Tue Sep 5 19:32:16 2023 -0700

    Merge branch 'main' into fix/864

commit 71166d0
Merge: bae04fa 71778ff
Author: Ethan Davidson <31261035+EthanThatOneKid@users.noreply.github.com>
Date:   Tue Jul 11 13:13:55 2023 -0700

    Merge branch 'main' into fix/864

commit bae04fa
Author: EthanThatOneKid <31261035+EthanThatOneKid@users.noreply.github.com>
Date:   Thu Jul 6 10:43:28 2023 -0700

    Add "Updating board member data"

    Edit "Custom workflows", add "Updating board member data"

commit 6157bd6
Author: EthanThatOneKid <31261035+EthanThatOneKid@users.noreply.github.com>
Date:   Sun Jul 2 12:31:33 2023 -0700

    Fix heading capitalization

commit 0ea38c2
Merge: eff5053 9dfdd48
Author: Ethan Davidson <31261035+EthanThatOneKid@users.noreply.github.com>
Date:   Sun Jul 2 12:07:05 2023 -0700

    Merge branch 'main' into fix/864

commit eff5053
Merge: 6b207a9 f6a5138
Author: Ethan Davidson <31261035+EthanThatOneKid@users.noreply.github.com>
Date:   Sun Jun 18 10:37:08 2023 -0700

    Merge branch 'main' into fix/864

commit 6b207a9
Author: Ethan Davidson <31261035+EthanThatOneKid@users.noreply.github.com>
Date:   Thu May 25 10:58:27 2023 -0700

    Update GETTING_STARTED.md

commit e75264f
Author: Ethan Davidson <31261035+EthanThatOneKid@users.noreply.github.com>
Date:   Thu May 25 10:56:11 2023 -0700

    Update GETTING_STARTED.md
@EthanThatOneKid
Copy link
Owner Author

Accidentally merged this PR early. Followed up in b857a01.

EthanThatOneKid added a commit that referenced this pull request Sep 7, 2023
@EthanThatOneKid
Drop support for legacy officer update automation (#903)
03c5869 
* Update GETTING_STARTED.md

* Update GETTING_STARTED.md

* Fix heading capitalization

* Add "Updating board member data"

Edit "Custom workflows", add "Updating board member data"

* Include periods on complete sentences

Resolves <#865 (comment)>.

* Fix "Getting started" title

* Getting started: clarify review cycle

Resolves <#865 (comment)>.

* CONTRIBUTING.md: add shortlinks and blog posts

* Remove legacy `update_officer.yaml` workflow
EthanThatOneKid added a commit that referenced this pull request Sep 7, 2023
@EthanThatOneKid
CONTRIBUTING.md typo
558ba83 
Related: #865.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@karnikaavelumani karnikaavelumani karnikaavelumani approved these changes

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

Successfully merging this pull request may close these issues.

Outdated documentation
2 participants
@EthanThatOneKid @karnikaavelumani