Skip to content

Minor Code Fixes #34

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 5 commits into from
Aug 11, 2020
Merged

Minor Code Fixes #34

merged 5 commits into from
Aug 11, 2020

Conversation

TheRealNate
Copy link
Collaborator

No description provided.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

ShiftLeft Scan Summary

Tool Critical High Medium Low Status
ShiftLeft Source code analyzer for JavaScript 0 0 4 4

Recommendation

Looks good 💯

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

ShiftLeft Scan Summary

Tool Critical High Medium Low Status
ShiftLeft Source code analyzer for JavaScript 0 0 4 4

Recommendation

Looks good 💯

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

ShiftLeft Scan Summary

Tool Critical High Medium Low Status
ShiftLeft Source code analyzer for JavaScript 0 0 4 4

Recommendation

Looks good 💯

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

ShiftLeft Scan Summary

Tool Critical High Medium Low Status
ShiftLeft Source code analyzer for JavaScript 0 0 4 4

Recommendation

Looks good 💯

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

ShiftLeft Scan Summary

Tool Critical High Medium Low Status
ShiftLeft Source code analyzer for JavaScript 0 0 4 4

Recommendation

Looks good 💯

@TheRealNate TheRealNate changed the title Remove unused assignment to options Minor Code Fixes Aug 11, 2020
@lgtm-com
Copy link

lgtm-com bot commented Aug 11, 2020

This pull request fixes 4 alerts when merging b9be7c8 into 20b499d - view on LGTM.com

fixed alerts:

  • 2 for Unused variable, import, function or class
  • 1 for Useless assignment to local variable
  • 1 for Unused or undefined state property

@TheRealNate TheRealNate merged commit d746265 into master Aug 11, 2020
@TheRealNate TheRealNate deleted the code-quality-fixes branch August 13, 2020 21:02
TheRealNate pushed a commit that referenced this pull request Aug 28, 2020
* Update MFA companion package (#27)

* Create manual.yml

* Create stale.yml

* Update stale.yml

* Update index.js

* Add a semicolon

* Remove unused assignment to options (#34)

* Remove unused assignment to options

* Mark SHA256 insufficient as wontfix

* Remove unused variable self

* Remove unused variable serializedValue

* Remove unused state properties loading
TheRealNate pushed a commit that referenced this pull request Dec 18, 2020
* Update MFA companion package

* Replace code for depreciated componentWillMount() (#10)

* Allow other AsyncStorage packages to be used

* Allows for option to specify AsyncStorage package...
Either React Native Async Storage (Expo), with fallback using @react-native-community/async-storage if no AsyncStorage is provided for backwards compatibility.

* Remove commented code

* Fix package version

* Add "Custom Storage Adapter" to Installation

* Update AsyncStorage options

* Bump package version to 2.0.2 since no longer a breaking change

* Throw an error instead of console error if no AsyncStorage installed

* Remove dead code `setItem()`

* Temporary solution to depreciated componentWillUpdate...
Using the [`UNSAFE_componentWillUpdate()`](https://reactjs.org/docs/react-component.html#unsafe_componentwillupdate) version to get throgh React v17.

* Moved code to a constructor to replace componentWillMount()...
depreciated code

Co-authored-by: Nathaniel Dsouza <nate@kidzideaz.xyz>

* Add useTracker function (#17)

* Bump dev package version

* 2.1.0-beta1

* Removing useTracker.js from dev since author has said it is not ready

* Remove useTracker

* Remove unnecessary code

* add useTracker and rewrite withTracker to use it (#31)

fixes #29

* Merge changes from master into dev (#32)

* Update MFA companion package (#27)

* Create manual.yml

* Merge updates from master into dev (#37)

* Update MFA companion package (#27)

* Create manual.yml

* Create stale.yml

* Update stale.yml

* Update index.js

* Add a semicolon

* Remove unused assignment to options (#34)

* Remove unused assignment to options

* Mark SHA256 insufficient as wontfix

* Remove unused variable self

* Remove unused variable serializedValue

* Remove unused state properties loading

* Bump package version

* Deprecate `Meteor.collection` (lowercase-C)

* Update Mongo Docs

* Update api.md

* Update Accounts Docs

* Update Tracker and Verbosity Docs

* Start adding table of contents

* Refactor to remove require cycle

* 2.1.0-beta3

* 2.1.0-beta4

* local: Automaticallly commit changes to storage after .insert, .update, or .remove

* Update User.js

* 2.1.0-beta5

* Refactor code to remove require cycles

* Bump Package Version to 2.1.0-rc1

* Delete shiftleft-analysis.yml

* Update package version for release

Co-authored-by: Polaris Web Technology <PolarisWT@users.noreply.github.com>
Co-authored-by: Sandeep Jain <sandeepjain@users.noreply.github.com>
Co-authored-by: EC2 Default User <ec2-user@ip-172-31-90-84.ec2.internal>
Co-authored-by: Kelly Copley <copleykj@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant