Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@adamralph
Copy link
Contributor

fixes #720

@adamralph adamralph changed the title green: added basic ScriptPacks acceptance test added basic ScriptPacks acceptance test Jul 13, 2014
@adamralph adamralph changed the title added basic ScriptPacks acceptance test ScriptPacks acceptance test Jul 13, 2014
@adamralph
Copy link
Contributor Author

_Fails on TravisCI_ 😢

@khellang
Copy link
Member

This also contains the commit from #796...

@adamralph
Copy link
Contributor Author

Yep, this one is on hold (see label). I was just experimenting and I wanted to have the proper error coming out of the installation step. Now that the other PR is merged I'll rebase and continue. At the moment, however, I'm not seeing a path out of the failure on Mono.

@adamralph
Copy link
Contributor Author

I have a suspicion it might be the absolute path to the local gallery for the script pack installation, i.e. /home/.... Changing to a relative path might fix it.

@khellang
Copy link
Member

Well that didn't work 😝

@adamralph
Copy link
Contributor Author

Rebased, but still need to fix things for the Mono build.

@adamralph adamralph self-assigned this Dec 17, 2014
@adamralph
Copy link
Contributor Author

Success at last! This is good to go now.

@adamralph adamralph removed the hold label Jan 3, 2015
@adamralph adamralph removed their assignment Jan 3, 2015
@khellang
Copy link
Member

khellang commented Jan 3, 2015

Awesome job @adamralph 😁

khellang added a commit that referenced this pull request Jan 3, 2015
ScriptPacks acceptance test
@khellang khellang merged commit dcfda00 into scriptcs:dev Jan 3, 2015
@adamralph adamralph deleted the 720 branch January 4, 2015 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Acceptance tests for using a script pack

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.