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

[SDK-1050] Supporting the new alt text feature in the WEB SDK #331

Merged
merged 3 commits into from
Nov 29, 2022

Conversation

pshoniuk
Copy link
Contributor

No description provided.

@pshoniuk pshoniuk self-assigned this Nov 29, 2022
@pshoniuk pshoniuk force-pushed the feat/SDK-1050 branch 2 times, most recently from 42ba090 to 45deb38 Compare November 29, 2022 14:54
@pshoniuk pshoniuk marked this pull request as ready for review November 29, 2022 15:47
skip_cleanup: true
on:
tags: true
Copy link
Collaborator

@giannif giannif Nov 29, 2022

Choose a reason for hiding this comment

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

@pshoniuk I believe the problem is that tags don't run on branch: master, as you can see in the previous commit I had this setting before trying tags: true

Unless you changed something else I missed, or Travis behavior has changed, this probably won't work

Edit: I see you removed the if, so maybe this will work 👍

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@giannif I think the root issue was this line. It just killed all the background processes, including some Travis agents, and thus the deployment was skipped. But to be sure, we need to check this assumption

Copy link
Collaborator

@giannif giannif left a comment

Choose a reason for hiding this comment

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

💃

LBBO added a commit to LBBO/giphy-js that referenced this pull request Nov 29, 2022
This is a desperate and experimental attempt to fix the pipeline,
since it works for that branch
LBBO added a commit to LBBO/giphy-js that referenced this pull request Nov 29, 2022
This is a desperate and experimental attempt to fix the pipeline,
since it works for that branch
@LBBO
Copy link

LBBO commented Nov 29, 2022

Sorry for the spam 😅 I was hoping the amended commit wouldn't appear here as well. Anyway, I'm just hoping that your changes to the travis config will fix my pipeline as well and somehow the whitespaces didn't survive my copy paste 🤷‍♂️

@giannif giannif merged commit 7814f8b into master Nov 29, 2022
@giannif giannif deleted the feat/SDK-1050 branch November 29, 2022 17:53
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.

3 participants