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

Bundle the captures inside a 'screenshot' folder #161

Closed
wants to merge 1 commit into from

Conversation

almet
Copy link
Contributor

@almet almet commented Apr 24, 2016

As discussed in #158

This won't work unless it's merged to master (links are done on the master branch).

@n1k0
Copy link
Collaborator

n1k0 commented Apr 24, 2016

Thanks. After discussion we're not gonna land this. On the other hand I've filed #162 about the intermittent failure you just got with this PR.

@n1k0 n1k0 closed this Apr 24, 2016
@n1k0 n1k0 deleted the screenshots-in-repo branch April 24, 2016 12:50
@almet
Copy link
Contributor Author

almet commented Apr 24, 2016

For later reference and other people reading: this will not land because it complexifies @n1k0 flow when he's working, as he's using a tool which directly uploads screenshots to imgur.

James-Brown98 pushed a commit to James-Brown98/react-jsonschema-form that referenced this pull request Feb 24, 2022
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.

None yet

2 participants