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

Change IDEA codestyle href #50

Merged
merged 1 commit into from Sep 15, 2020
Merged

Change IDEA codestyle href #50

merged 1 commit into from Sep 15, 2020

Conversation

Persdre
Copy link
Contributor

@Persdre Persdre commented Sep 15, 2020

On the "Setting up and getting started" page of AB3 developer guide (https://nus-cs2103-ay2021s1.github.io/tp/SettingUp.html), the href for "[se-edu/guides] IDEA: Configuring the JDK" is pointed towards https://se-education.org/guides/tutorials/checkstyle.html, instead of https://se-education.org/guides/tutorials/intellijCodeStyle.html

@canihasreview
Copy link

canihasreview bot commented Sep 15, 2020

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

See this repository's contribution guide for more information.

@codecov-commenter
Copy link

Codecov Report

Merging #50 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #50   +/-   ##
=========================================
  Coverage     72.31%   72.31%           
  Complexity      400      400           
=========================================
  Files            70       70           
  Lines          1228     1228           
  Branches        124      124           
=========================================
  Hits            888      888           
  Misses          308      308           
  Partials         32       32           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6389d85...df7ded4. Read the comment docs.

@damithc
Copy link
Contributor

damithc commented Sep 15, 2020

Thanks for the fix @Persdre 💯

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.

None yet

3 participants