Change from ATF to OMB content #7

Merged
merged 4 commits into from Jan 23, 2017

Conversation

Projects
None yet
7 participants
@theresaanna
Member

theresaanna commented Jan 19, 2017

Changes the content of the README from ATF to OMB-specific. Removes Gemnasium temporarily as I am unable to configure the project due to our Github org breaking the Gemnasium site.

Change from ATF to OMB content
Changes the content of the README from ATF to OMB-specific. Removes Gemnasium temporarily as I am unable to configure the project due to our Github org breaking the Gemnasium site.
@theresaanna

This comment has been minimized.

Show comment
Hide comment
@theresaanna

theresaanna Jan 19, 2017

Member

Need to get Travis passing

Member

theresaanna commented Jan 19, 2017

Need to get Travis passing

@theresaanna theresaanna reopened this Jan 20, 2017

@theresaanna theresaanna requested a review from cmc333333 Jan 20, 2017

@theresaanna

This comment has been minimized.

Show comment
Hide comment
@theresaanna

theresaanna Jan 20, 2017

Member

And now Travis is passing! Though we will want to remove my temporary script command as soon as we have something to run.

Member

theresaanna commented Jan 20, 2017

And now Travis is passing! Though we will want to remove my temporary script command as soon as we have something to run.

@stvnrlly

This comment has been minimized.

Show comment
Hide comment
@stvnrlly

stvnrlly Jan 22, 2017

Member

I stumbled into this PR somehow and saw the Gemnasium thing. Until that gets figured out, you can still activate the project with some jQuery: $('[name=submit_github_projects]').before('<input type="hidden" name="full_names[]" value="org/repo-name" />').click().

(via https://gsa-tts.slack.com/archives/infrastructure/p1484672271001423)

Member

stvnrlly commented Jan 22, 2017

I stumbled into this PR somehow and saw the Gemnasium thing. Until that gets figured out, you can still activate the project with some jQuery: $('[name=submit_github_projects]').before('<input type="hidden" name="full_names[]" value="org/repo-name" />').click().

(via https://gsa-tts.slack.com/archives/infrastructure/p1484672271001423)

@jipiboily

This comment has been minimized.

Show comment
Hide comment
@jipiboily

jipiboily Jan 23, 2017

Hey, JP from Gemnasium here. Did you contact our support @theresaanna? We're always happy to help 😀

Hey, JP from Gemnasium here. Did you contact our support @theresaanna? We're always happy to help 😀

@cmc333333

This comment has been minimized.

Show comment
Hide comment
@cmc333333

cmc333333 Jan 23, 2017

Member

Hey there @stvnrlly, we did indeed add the project through the JS work around. @jipiboily, I believe 18F's already contacted your support group (search for messages from @NoahKunin) -- effectively, we have too many repositories in our Github org.

In any event, the work around works! Thanks @theresaanna !

Member

cmc333333 commented Jan 23, 2017

Hey there @stvnrlly, we did indeed add the project through the JS work around. @jipiboily, I believe 18F's already contacted your support group (search for messages from @NoahKunin) -- effectively, we have too many repositories in our Github org.

In any event, the work around works! Thanks @theresaanna !

@cmc333333 cmc333333 merged commit ec0325d into master Jan 23, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@cmc333333 cmc333333 referenced this pull request in 18F/before-you-ship Jan 23, 2017

Merged

Add instructions around Gemnasium #248

@afeld

This comment has been minimized.

Show comment
Hide comment
@afeld

afeld Jan 23, 2017

Member

@jipiboily We wrote up some workarounds in 18F/before-you-ship#248 - would love input there!

Member

afeld commented Jan 23, 2017

@jipiboily We wrote up some workarounds in 18F/before-you-ship#248 - would love input there!

@jipiboily

This comment has been minimized.

Show comment
Hide comment
@jipiboily

jipiboily Jan 25, 2017

@afeld sorry for the delay here. Someone's going to look into it shortly.

@afeld sorry for the delay here. Someone's going to look into it shortly.

@cmc333333 cmc333333 deleted the theresaanna-patch-2 branch Jan 25, 2017

@gonzoyumo

This comment has been minimized.

Show comment
Hide comment
@gonzoyumo

gonzoyumo Jan 26, 2017

Hi there, Olivier from Gemnasium.

There was indeed a support request open by the account owner. At that time the issue was solved with a manual sync to work around the UI timeout.

I just did it again but the 18F org does no longer show up as an accessible org for this Github account.

Can someone open an issue on support.gemnasium.com to follow up and talk about it?

Thanks and nice work around btw ;)

Hi there, Olivier from Gemnasium.

There was indeed a support request open by the account owner. At that time the issue was solved with a manual sync to work around the UI timeout.

I just did it again but the 18F org does no longer show up as an accessible org for this Github account.

Can someone open an issue on support.gemnasium.com to follow up and talk about it?

Thanks and nice work around btw ;)

@gonzoyumo

This comment has been minimized.

Show comment
Hide comment
@gonzoyumo

gonzoyumo Jan 27, 2017

@NoahKunin thanks, we'll try to work on that. Anyway as I mentioned above it won't totally resolve your issue. The associated Github account (@dlapiduz) doesn't have 18F in its orgs list anymore.

Please open a ticket by clicking on "Contact support" on http://support.gemnasium.com so that we can handle that.

@NoahKunin thanks, we'll try to work on that. Anyway as I mentioned above it won't totally resolve your issue. The associated Github account (@dlapiduz) doesn't have 18F in its orgs list anymore.

Please open a ticket by clicking on "Contact support" on http://support.gemnasium.com so that we can handle that.

toolness added a commit that referenced this pull request Dec 11, 2017

Merge pull request #7 from 18F/paragraphs
Infer paragraph semantics
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment