Configure WhiteSource Bolt for GitHub #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![mend-bolt-for-github[bot]](https://avatars.githubusercontent.com/in/16809?s=60&v=4){kind=small}
PowerShell#346) * added new RegisterPSResourceRepository.cs with parameters so far * add PSRepositoryItemInfo * added passthru, need to complete implementation * added code and tests for REgister-PSResourceRepository * add code for Get-PSResourceRepository and move to srcNew * add files to srcNew * moved cmdlets and added Set-PSResourceRepository cmdlet * removed set, get, unregister resource repository cmdlets to put in later PRs * rename Register-PSResourceRepository cmdlet and references to previous name of cmdlet * added comemnts where needed * revert changes accidentally made * add Get-PSResourceRepository, Unregister-PSResourceRepository so tests for Register can work * reset changes to settings.json * revert unintended changes to srcOld folder * revert unintended changes to testOld * revert unintended changes to PowerShellGet.psd1 script * add EOL to some files that were missing it * remove commented out code from PSGetTestUtils.cs * changed using statements so that namespace and class name can have same names without error * fix reference in ArgumentCompleter.cs * fix some codacy warnings * make RepositorySettings.cs a static class and update rest of classes that use it * remove not so helpful comments * explicitly type variables that were previously implicitly typed as var * remove unintended writeDebug message * use yield to return IEnumerable for ArgumentCompleter and remove private helper method * remove unncessary parameter set names from cmdlets which don't have multiple parameter sets * store parameter set names as const fields * remove unused RemotingCapability field from Cmdlets * RepositorySettings and PSRepositoryItem now belong to UtilClasses namespace and removed older using statements * replace private fields with Properties, use switch case for parameter set names * Remove comments * create helper method to search if repo with name already exists * change error handling to be non terminating where needed, and continue looping through rest of repositories if error arises * remove some unneccessary comments * refactor so there's a helper method calling the Add() API * call Read() to find existing repo by name in Add() * make commented description of Add() clearer * refactor FindRepositoryXML() * throw error if non implemented parameters are attempted to be used * refactor CheckRepositoryStore() and how/where it throws errors * wrap RepositorySettings file handling code with try/catch * minor fix * remove unneccessary using statements * Delete UnregisterPSResourceRepository.Tests.ps1 * include error specific tests for Register-PSResourceRepository * change Repositories parameter type from List<Hashtable> to Hashtable[] * add tests for Unregister and refactor RepositorySettings to handle Unregister related errors * add out errorMsgs to RepositorySettings Read() and refactor GetPSResourceRepository and its tests * fix ArgumentCompleter issue * remove uneccesary comments from RepositorySettings * rename FindRepositoryElement helper method * refactor FindRepositoryElement code to just return and be more readable * styling with if keywords * fix indentation and bracket placement according to PowerShell style standards * added Debug statements to register * added try catch to Get and Unregister, changed out string[] errorMsgs syntax * fix Debug statements and remove unneccessary try catch * remove shouldSupportProcess from Get * put check for repo store in BeginProcessing() and change default switch case to use dbg assert * check if repo name is whitespace * correct error message typo * remove unncessary comment * fix typo in tests, rename errorMsgs to errorList * make PSRepositoryItem sealed class, add ShouldProcess() support to Register * add ShouldSupportProcess = true to Unregister * don't write error if names matching wildcard aren't found * not set errorList to null as not needed * add powershell helper to add quotes to ArgumentCompleter to deal with Name with spaces * tests use now * changed Get and Unregister tests to use FullyQualifiedErrorId * remove setters from PSRepositoryItem * remove ValueByPipeline tag from Proxy, ProxyCredential and add onto Repositories * use ErrorId for Unregister tests * all tests now rely on FullyQualifiedErrorId * make shouldsupport() target action message more intuitive * fix target name formatting for ShouldProcess() * add newline after brackets and #endregion per powershell style guidelines * add newline in Utils.cs * trim trailing and leading whitespaces from Name and throw error if Name is just whitespace * add test for throwing error if Name is whitespace for register * rename Get and Unregister to remove hype in cmdlet name * fix codacy warning * fixed some naming conventions in test to fix codacy warnings * renamed PSRepositoryItem -> PSRepositoryInfo to match Amber's output class naming * Update Name parameter description Update name parameter description to say it now supports wildcards in Name * fix styling issues * change Repository settings fields to private * change testcases syntax to Pester v4 syntax * change temp variable name, update PSResourceRepository.xml file comment * add newline to end of Utils.cs * organize using directive statements alphabetically * change error type for checkRepositoryStore error thrown
…werShell#355) * add help docs for unregister, register, get -psresourcerepository cmdlets * fix synopsis for Register help doc * minor formatting issue * resolve current user related feedback in Get-PSResourceRepository.md * changes powershell modules to powershell resources * resolved feedback to Register-PSResourceRepository.md * resolve feedback on Unregister-PSResourceRepository.md * remove current user references * change description for Repositories param * fix codacy issue
WIP: Design docs
…data (PowerShell#361) * Add TryPSGetInfo utility function * Fix Codacy and test issues * Fix DateTime tests * Rename method * Add TryWritePSGetInfo utility method * Fix Codacy issue * Small syntax change * Address CR comments
* Move the PSResourceInfo class to its own source file, and rename test * Add missing copyright to new source file * Fix Codeacy warning
* added implemenation and test for Set * remove unnecessary comments * add some summary parameter comments * remove more comments * remove NameParameterSetHelper() * add test and code for detecting if name key in repositories param is null * update help doc for Set * fix codacy issues * add Debug statements * add newline at end of SetPSResourceRepository.cs file * used parameterset name const variables instead of parameter set strings * remove Credential parameter * remove Proxy and ProxyCredential parameters * remove Credential parameter from help docs as well * trim whitespace with Trim(), no param needed * order directives alphabetically * remove Priority and URL parameters from pipeline
Co-authored-by: Anam Navied <[email protected]> Co-authored-by: Amber Erickson <[email protected]>
refactor Find-PSResource with core functionality
…e and test directories (PowerShell#382)
* Add -SkipDependencyCheck to install/save/update cmdlets * Change -Force to -SkipDependencyCheck * Fix tests
Add progress bar for install
…#556) add PassThru to Unregister-PSResourceRepository cmdlet
install should not fail silently
clean up tests and add tests for certain errors
* Update change log for beta 12 release * Fix progress bar and test issues
* for errors being written in Save write to a variable * for Uninstall, fix errors being written * add more verification that Install step also worked
add support for credential persistence
Rename params *url to *uri
Add support for Uninstall -Prerelease parameter uninstalling only prerelease versions
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). For more information, open the CLA check for this pull request. |
classicvalues
force-pushed
the
master
branch
2 times, most recently
from
d1cfda8 to
95727a9
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Welcome to WhiteSource Bolt for GitHub! This is an onboarding PR to help you understand and configure settings before WhiteSource starts scanning your repository for security vulnerabilities.
🚦 WhiteSource Bolt for GitHub will start scanning your repository only once you merge this Pull Request. To disable WhiteSource Bolt for GitHub, simply close this Pull Request.
What to Expect
This PR contains a '.whitesource' configuration file which can be customized to your needs. If no changes were applied to this file, WhiteSource Bolt for GitHub will use the default configuration.
Before merging this PR, Make sure the Issues tab is enabled. Once you merge this PR, WhiteSource Bolt for GitHub will scan your repository and create a GitHub Issue for every vulnerability detected in your repository.
If you do not want a GitHub Issue to be created for each detected vulnerability, you can edit the '.whitesource' file and set the 'minSeverityLevel' parameter to 'NONE'.
❓ Got questions? Check out WhiteSource Bolt for GitHub docs.
If you need any further assistance then you can also request help here.