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

Ag pasture dev #3727

Closed
wants to merge 18 commits into from
Closed

Conversation

mpandreucci
Copy link

working on #3689

@hol353
Copy link
Contributor

hol353 commented Apr 11, 2019

Good work @mpandreucci. It ran green. But...
Can you remove the PDF files? We don't commit them.

@mpandreucci
Copy link
Author

mpandreucci commented Apr 11, 2019 via email

@mpandreucci
Copy link
Author

@hol353 how do I take the pdf file out of it? I asked Laura but she didn't know how to do it through source tree. Do I need to delete it from my prototypes folder?

@jbrider
Copy link
Contributor

jbrider commented Apr 11, 2019

@mpandreucci in Sourcetree, there should be a 'stop tracking' option when right clicking on the file in the staging window - that would be the easiest way.
Alternatively - if you remove the file from the folder it should show up in Sourcetree as removed - you can then commit that file as being removed.

@mpandreucci
Copy link
Author

Hi @jbrider . I'll have a go at removing it and then committing the change because I couldn't find out how to 'Stop tracking' it.
Thanks for that.

@hol353
Copy link
Contributor

hol353 commented Apr 11, 2019

@mpandreucci :There are still 2 pdf files in here. Look at the 'Files changed' tab in the pull request.

@mpandreucci
Copy link
Author

I'll delete them too (not sure how they ended up there since they have nothing to do with agpasture). Sorry about that

hol353 and others added 3 commits April 12, 2019 08:50
@mpandreucci mpandreucci deleted the AgPastureDev branch April 12, 2019 01:54
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