Skip to content
This repository has been archived by the owner on Nov 1, 2024. It is now read-only.

Make sdkPath a getter #100

Merged
merged 12 commits into from
Apr 11, 2024
Merged

Make sdkPath a getter #100

merged 12 commits into from
Apr 11, 2024

Conversation

lrhn
Copy link
Contributor

@lrhn lrhn commented Apr 3, 2024

Add sdkPath getter and deprecate getSdkPath function.

Fixes dart-lang/tools#853

Add `sdkPath` getter and deprecate `getSdkPath` function.
@lrhn lrhn requested a review from kevmoo April 3, 2024 08:49
lrhn added 8 commits April 3, 2024 10:51
Update tests to not use deprecated name.
Increment version to 0.4.2-dev.
Break long line. 
(Editing directly in GitHub, so no formatter.)
Clean up environment access, make all uses of exception pass only the environment entry name.
Long line in comment, not caught by formatter.
Copy link

github-actions bot commented Apr 3, 2024

Package publishing

Package Version Status Publish tag (post-merge)
package:cli_util 0.4.2-dev ready to publish v0.4.2-dev

Documentation at https://github.com/dart-lang/ecosystem/wiki/Publishing-automation.

@kevmoo kevmoo requested review from devoncarew and removed request for kevmoo April 3, 2024 16:51
Copy link
Contributor

@devoncarew devoncarew left a comment

Choose a reason for hiding this comment

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

lgtm; I'd switch the package verison to -wip however. We can drop the extension and publish this once this version rolls into the sdk + google3.

lib/cli_util.dart Show resolved Hide resolved
pubspec.yaml Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
lrhn and others added 2 commits April 11, 2024 08:47
Co-authored-by: Devon Carew <[email protected]>
Co-authored-by: Devon Carew <[email protected]>
@lrhn lrhn merged commit e222c56 into master Apr 11, 2024
5 of 6 checks passed
@lrhn lrhn deleted the lrhn-patch-sdkPath branch April 11, 2024 06:54
copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request Apr 11, 2024
…source_map_stack_trace, source_maps

Revisions updated by `dart tools/rev_sdk_deps.dart`.

args (https://github.com/dart-lang/args/compare/b2e2224..5c83bc9):
  5c83bc9  2024-04-11  Devon Carew  rev to 2.5.0 and prep to publish (dart-archive/args#267)

cli_util (https://github.com/dart-lang/cli_util/compare/12cd216..e222c56):
  e222c56  2024-04-11  Lasse R.H. Nielsen  Make `sdkPath` a getter (dart-archive/cli_util#100)

ecosystem (https://github.com/dart-lang/ecosystem/compare/de03da1..95fed18):
  95fed18  2024-04-09  Kevin Moore  dart_flutter_team_lints: bump pkg:checks dev dep (dart-lang/ecosystem#250)

http (https://github.com/dart-lang/http/compare/caad9ca..b7477b1):
  b7477b1  2024-04-10  Hossein Yousefi  Upgrade jni and jnigen to 0.8.0 (dart-lang/http#1176)

mockito (https://github.com/dart-lang/mockito/compare/3ef744f..81ecb88):
  81ecb88  2024-04-11  Ilya Yanok  Fix one README example that were broken by dart-lang/mockito@1dcd822 Also sync the executable version of it with the README.

package_config (https://github.com/dart-lang/package_config/compare/486cc4b..854dc19):
  854dc19  2024-04-11  Lasse R.H. Nielsen  Fix typos. (dart-lang/package_config#147)

source_map_stack_trace (https://github.com/dart-lang/source_map_stack_trace/compare/d03fd9b..9ed8875):
  9ed8875  2024-04-10  Kevin Moore  Update the latest version of lints (dart-archive/source_map_stack_trace#50)
  5f7f3a7  2024-04-09  Kevin Moore  blast_repo fixes (dart-archive/source_map_stack_trace#49)

source_maps (https://github.com/dart-lang/source_maps/compare/64d07fa..aedfc39):
  aedfc39  2024-04-09  Kevin Moore  blast_repo fixes (dart-lang/source_maps#89)
  45a66c3  2024-04-09  Kevin Moore  Bump lints, require Dart 3.3 (dart-lang/source_maps#88)

Change-Id: Ibe7c030b2b7377ecb5fe81d05ae1df605da1ec6a
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/362193
Commit-Queue: Konstantin Shcheglov <[email protected]>
Reviewed-by: Konstantin Shcheglov <[email protected]>
Auto-Submit: Devon Carew <[email protected]>
mosuem pushed a commit to dart-lang/tools that referenced this pull request Oct 25, 2024
Add `sdkPath` getter and deprecate `getSdkPath` function.

Update tests to not use deprecated name.

Clean up environment access, make all uses of exception pass only the environment entry name.

Increment version to 0.4.2-wip
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

getSdkPath should be a getter, and not named get
2 participants