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

[Revised] Adding MarkBind as an alternative docs option #206

Closed
wants to merge 1 commit into from

Conversation

damithc
Copy link
Contributor

@damithc damithc commented Sep 11, 2023

A revised version based on #156
Once finalized, all commits will be squashed to one commit and the commit message will mention #156 as the primary source of changes
Deployed site https://damithc.github.io/ab3-markbind/index.html

@canihasreview
Copy link

canihasreview bot commented Sep 11, 2023

Click here to submit a new iteration when this PR is ready for review.

See this repository's contribution guide for more information.

@codecov
Copy link

codecov bot commented Sep 11, 2023

Codecov Report

Merging #206 (da647a3) into master (517b785) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #206   +/-   ##
=========================================
  Coverage     75.26%   75.26%           
  Complexity      419      419           
=========================================
  Files            71       71           
  Lines          1338     1338           
  Branches        126      126           
=========================================
  Hits           1007     1007           
  Misses          301      301           
  Partials         30       30           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@damithc
Copy link
Contributor Author

damithc commented Sep 11, 2023

@tlylt can have a look?

token: ${{ secrets.GITHUB_TOKEN }}
rootDirectory: './docs'
baseUrl: '/addressbook-level3' # replace with your repo name
version: '^5.0.2'
Copy link

Choose a reason for hiding this comment

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

Suggested change
version: '^5.0.2'
version: '^5.1.0'

@tlylt
Copy link

tlylt commented Sep 11, 2023

@tlylt can have a look?

Other than the above, LGTM!

Let's migrate the docs site from Jekyll to MarkBind.

Primary author: @tlylt in /pull/156
Further tweaks: @damithc in /pull/206
damithc pushed a commit to damithc/ab3-markbind that referenced this pull request Sep 11, 2023
Let's migrate the docs site from Jekyll to MarkBind.

Primary author: @tlylt in se-edu/addressbook-level3/pull/156
Further tweaks: @damithc in se-edu/addressbook-level3/pull/206

(cherry picked from commit da647a31d3f265287dbaf3cac6adc8a7272be68c)
@damithc
Copy link
Contributor Author

damithc commented Sep 12, 2023

Released to students as an option (see step 4 here)
Closing this PR as it is never meant to be merged. We can observe how things go this semester and revise further in the following semester.

Thanks again @tlylt for your immense help with this. 💯

@damithc damithc closed this Sep 12, 2023
lihongguang00 pushed a commit to lihongguang00/tp that referenced this pull request Oct 14, 2023
Let's migrate the docs site from Jekyll to MarkBind.

Primary author: @tlylt in se-edu/addressbook-level3/pull/156
Further tweaks: @damithc in se-edu/addressbook-level3/pull/206
damithc pushed a commit that referenced this pull request Feb 9, 2024
Let's migrate the docs site from Jekyll to MarkBind.

Primary author: @tlylt in /pull/156
Further tweaks: @damithc in /pull/206
xuelinglow pushed a commit to xuelinglow/tp that referenced this pull request Feb 26, 2024
Let's migrate the docs site from Jekyll to MarkBind.

Primary author: @tlylt in se-edu/addressbook-level3/pull/156
Further tweaks: @damithc in se-edu/addressbook-level3/pull/206
damithc pushed a commit to damithc/ab3-markbind that referenced this pull request Aug 8, 2024
Let's migrate the docs site from Jekyll to MarkBind.

Primary author: @tlylt in se-edu/addressbook-level3/pull/156
Further tweaks: @damithc in se-edu/addressbook-level3/pull/206
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.

2 participants