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

4567 - Pointing datagovtheme to datalayer pr to test GTM ingestion #1308

Closed
wants to merge 33 commits into from

Conversation

dlennox24
Copy link
Contributor

FuhuXia and others added 30 commits March 29, 2024 13:36
update dependencies, update snyk ignore list
update snyk expirations and remove outdated
4567- Remove all instances of ckanext-googleanalytics
[Snyk + GH Actions] Update requirements
[Snyk + GH Actions] Update requirements
@dlennox24 dlennox24 self-assigned this Apr 15, 2024
@FuhuXia
Copy link
Member

FuhuXia commented Apr 15, 2024

No need to create PR to change the develop branch. You can just overwrite it.

@FuhuXia FuhuXia closed this Apr 15, 2024
@jbrown-xentity
Copy link
Contributor

@dlennox24 this branch isn't protected, you can use something like https://stackoverflow.com/a/5740047 to push your local onto the develop branch. Best practice is to post a notice on slack for other team members to know what's going on, in case they are trying to deploy changes as well.

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.

7 participants