Skip to content

chore: remove catalina support #800

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

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Conversation

shakyShane
Copy link
Collaborator

@shakyShane shakyShane commented Apr 28, 2025

Reviewer:
Asana: https://app.asana.com/1/137249556945/project/1198964220583541/task/1210068517547285?focus=true

Description

POC for removing Catalina support

Steps to test

@shakyShane
Copy link
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Signed-off-by: Emanuele Feliziani <[email protected]>
… shane/catalina-support

Signed-off-by: Emanuele Feliziani <[email protected]>

# Conflicts:
#	dist/autofill-debug.js
#	dist/autofill.js
… shane/catalina-support

Signed-off-by: Emanuele Feliziani <[email protected]>

# Conflicts:
#	dist/autofill-debug.js
#	dist/autofill.js
@GioSensation
Copy link
Member

@shakyShane Yo! I saw this in the list and merged the latest from main and removed one failing test. The test was a Catalina-specific test and since we've removed all that it was not needed (and of course it was failing, the feature is no longer there). Were you planning anything else here?

@borgateo
Copy link
Contributor

borgateo commented Jul 3, 2025

Hey @shakyShane are you planning to continue working on this PR?
Any objections if I close it?

@dbajpeyi
Copy link
Collaborator

Taking this over from Shane (discussed on MM).

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.

4 participants