Skip to content
This repository was archived by the owner on Jan 5, 2026. It is now read-only.

chore: [#3518] Remove public access modifier from adaptive-expressions - Part 3#4226

Merged
tracyboehrer merged 2 commits intomainfrom
southworks/delete/public-adaptive-expressions-3
Jun 16, 2022
Merged

chore: [#3518] Remove public access modifier from adaptive-expressions - Part 3#4226
tracyboehrer merged 2 commits intomainfrom
southworks/delete/public-adaptive-expressions-3

Conversation

@sw-joelmut
Copy link
Collaborator

Addresses #3518
#minor

Description

This PR removes the public access modifier from all constructors, methods and properties for the adaptive-expressions/builtinFunctions folder.

Note: These changes do not include the Typescript Constructor Shorthands (link).

Specific Changes

  • Removes all public access modifiers from where it's declared under the adaptive-expressions/builtinFunctions folder.

@sw-joelmut sw-joelmut requested a review from a team as a code owner May 2, 2022 14:25
@coveralls
Copy link

coveralls commented May 2, 2022

Pull Request Test Coverage Report for Build 2258486226

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.003%) to 84.48%

Totals Coverage Status
Change from base Build 2245574340: -0.003%
Covered Lines: 19902
Relevant Lines: 22310

💛 - Coveralls

@tracyboehrer tracyboehrer merged commit ee9e631 into main Jun 16, 2022
@tracyboehrer tracyboehrer deleted the southworks/delete/public-adaptive-expressions-3 branch June 16, 2022 13:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants