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

Java path doc update for Mac OS #2910

Merged
merged 3 commits into from
Feb 1, 2021
Merged

Java path doc update for Mac OS #2910

merged 3 commits into from
Feb 1, 2021

Conversation

lcampos
Copy link
Contributor

@lcampos lcampos commented Feb 1, 2021

What does this PR do?

Adds a note on how to get the Java path for Mac OS. Original PR #2899.

What issues does this PR fix or reference?

@W-8836836@

mikeflemingcfs and others added 2 commits February 1, 2021 15:09
I wasted a lot of time using the suggested paths, and could even see them in my Mac Finder app.  However, after much Googling the terminal command that found me, my true Java_Home was 'Which Java', cutting and pasting this path into my VSC settings overcame the settings error I kept receiving when selecting an Apex class.
@lcampos lcampos requested a review from a team as a code owner February 1, 2021 23:14
@codecov
Copy link

codecov bot commented Feb 1, 2021

Codecov Report

Merging #2910 (c371924) into develop (e164537) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #2910   +/-   ##
========================================
  Coverage    75.73%   75.73%           
========================================
  Files          277      277           
  Lines        10638    10638           
  Branches      1230     1230           
========================================
  Hits          8057     8057           
  Misses        2239     2239           
  Partials       342      342           

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 e164537...c371924. Read the comment docs.

Copy link
Contributor

@ekapner ekapner left a comment

Choose a reason for hiding this comment

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

capitalize Terminal

Co-authored-by: Emily Kapner <ekapner@salesforce.com>
@lcampos lcampos merged commit 3603c4e into develop Feb 1, 2021
@lcampos lcampos deleted the javapath_macos branch February 1, 2021 23:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants