Skip to content
This repository has been archived by the owner on Apr 25, 2023. It is now read-only.

fix: password policy conversion in config #113

Merged
merged 5 commits into from
Oct 28, 2021
Merged

fix: password policy conversion in config #113

merged 5 commits into from
Oct 28, 2021

Conversation

KaWaite
Copy link
Member

@KaWaite KaWaite commented Oct 28, 2021

Fixes a problem in how the config file's password policy is converted from string to RegExp. Helped by @rot1024

@netlify
Copy link

netlify bot commented Oct 28, 2021

✔️ Deploy Preview for reearth-web ready!

🔨 Explore the source changes: 59e4126

🔍 Inspect the deploy log: https://app.netlify.com/sites/reearth-web/deploys/617a2132115aae00084ca63d

😎 Browse the preview: https://deploy-preview-113--reearth-web.netlify.app

@codecov
Copy link

codecov bot commented Oct 28, 2021

Codecov Report

Merging #113 (59e4126) into main (22e0acd) will increase coverage by 0.10%.
The diff coverage is 75.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #113      +/-   ##
==========================================
+ Coverage   44.71%   44.81%   +0.10%     
==========================================
  Files          55       56       +1     
  Lines        1087     1100      +13     
  Branches      170      174       +4     
==========================================
+ Hits          486      493       +7     
- Misses        549      554       +5     
- Partials       52       53       +1     
Impacted Files Coverage Δ
src/config.ts 53.84% <75.00%> (ø)

src/config.test.ts Outdated Show resolved Hide resolved
src/config.test.ts Show resolved Hide resolved
rot1024
rot1024 previously approved these changes Oct 28, 2021
src/config.test.ts Outdated Show resolved Hide resolved
@rot1024 rot1024 changed the title fix: passpolicy conversion in config fix: password policy conversion in config Oct 28, 2021
Copy link
Member

@HideBa HideBa left a comment

Choose a reason for hiding this comment

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

LGTM

@KaWaite KaWaite merged commit 5d57c4f into main Oct 28, 2021
@KaWaite KaWaite deleted the fix/password-val branch October 28, 2021 05:57
keiya01 pushed a commit that referenced this pull request Apr 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants