Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Run password rules workflow on schedule #334

Merged
merged 1 commit into from
Jun 22, 2023

Conversation

alistairjcbrown
Copy link
Member

@alistairjcbrown alistairjcbrown commented Jun 21, 2023

Reviewer: @shakyShane
Asana: https://app.asana.com/0/1198964220583541/1204875111329300/f

Description

Move password rules generation to schedule and PR

Steps to test

Test run created PR -- #335

@alistairjcbrown alistairjcbrown force-pushed the abrown/update-password-rules-workflow branch 3 times, most recently from 188a42a to c0e7ece Compare June 21, 2023 18:37
@alistairjcbrown alistairjcbrown force-pushed the abrown/update-password-rules-workflow branch from 46bc6a0 to 49407d5 Compare June 21, 2023 18:54
Copy link
Collaborator

@shakyShane shakyShane left a comment

Choose a reason for hiding this comment

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

Thanks, much better 🙏🏻

echo "currentDate=$(date +'%Y-%m-%d')" >> "$GITHUB_OUTPUT"
- name: Create PR for updated rules
if: ${{ steps.changes.outputs.hasChanged == 'true' }}
uses: peter-evans/create-pull-request@88bf0de51c7487d91e1abbb4899332e602c58bbf
Copy link
Collaborator

Choose a reason for hiding this comment

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

I was going to question the use of this - but then I realised it's used in the autofill automation already :)

Copy link
Member Author

Choose a reason for hiding this comment

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

@shakyShane Yeah, just copy-pasted the one we're using for the release flow 😁 -- I assume it's the version that's been checked and that we're good to use.

@alistairjcbrown alistairjcbrown merged commit f49b163 into main Jun 22, 2023
3 checks passed
@alistairjcbrown alistairjcbrown deleted the abrown/update-password-rules-workflow branch June 22, 2023 09:55
CDRussell pushed a commit to duckduckgo/Android that referenced this pull request Jul 14, 2023
Task/Issue URL:
https://app.asana.com/0/1205060095332912/1205060095332912
Autofill Release:
https://github.com/duckduckgo/duckduckgo-autofill/releases/tag/8.0.0


## Description
Updates Autofill to version
[8.0.0](https://github.com/duckduckgo/duckduckgo-autofill/releases/tag/8.0.0).

### Autofill 8.0.0 release notes
## What's Changed
* Remove in-context eligible pixel by @alistairjcbrown in
duckduckgo/duckduckgo-autofill#330
* Run password rules workflow on schedule by @alistairjcbrown in
duckduckgo/duckduckgo-autofill#334
* Add build step after pulling password rules by @alistairjcbrown in
duckduckgo/duckduckgo-autofill#338
* Update password rules (2023-06-23) by @daxmobile in
duckduckgo/duckduckgo-autofill#341
* Update password rules (2023-06-27) by @daxmobile in
duckduckgo/duckduckgo-autofill#345
* tests: attach JSON mocks as attachments by @shakyShane in
duckduckgo/duckduckgo-autofill#348
* tests: decouple 'login helpers' from navigating to separate tests by
@shakyShane in
duckduckgo/duckduckgo-autofill#350
* Couple more fixes by @GioSensation in
duckduckgo/duckduckgo-autofill#351
* fix: Typescript bug in tests + removed duplication within in-context
tests by @shakyShane in
duckduckgo/duckduckgo-autofill#352
* Testing updates by @shakyShane in
duckduckgo/duckduckgo-autofill#356
* Update password rules (2023-07-12) by @daxmobile in
duckduckgo/duckduckgo-autofill#357
* feat: email protection saving by @shakyShane in
duckduckgo/duckduckgo-autofill#355

## New Contributors
* @daxmobile made their first contribution in
duckduckgo/duckduckgo-autofill#341

**Full Changelog**:
duckduckgo/duckduckgo-autofill@7.2.0...8.0.0

## Steps to test
This release has been tested during autofill development. For smoke test
steps see [this
task](https://app.asana.com/0/1198964220583541/1200583647142330/f).

Co-authored-by: shakyShane <shakyShane@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants