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

Added 250 Military AC #490

Closed
wants to merge 6 commits into from
Closed

Conversation

g0Idfinger
Copy link
Contributor

Describe your changes

PLANE INFO: Info pulled from my VRS DB since Jan 1,2024

Info pulled from my VRS DB since Jan 1,2024
@github-actions github-actions bot added the planes Plane info label Mar 20, 2024
@github-actions github-actions bot added the images Plane images label Mar 21, 2024
@dziban303
Copy link
Collaborator

is this done? trying to figure out why it ran the derivative script before merging

@g0Idfinger
Copy link
Contributor Author

I was going to ask how you ran that on my side but not your side?

@dziban303
Copy link
Collaborator

dziban303 commented Mar 22, 2024

I didn't run anything. I think you have to disable the scripts on your end but @Phaeton is more familiar with it than me. I'm not sure if it'll affect a merge or not.

I seem to screw something up when I do anything other than my own routine, I should be kept far away from github

@g0Idfinger
Copy link
Contributor Author

I thought I just ran the scripts locally to ensure there were no dupes or csv formating errors before my commit. I too, know enough to be dangerous and love to tinker. :P

@Phaeton
Copy link
Collaborator

Phaeton commented Mar 22, 2024

@g0Idfinger In the future, make your changes in a branch, not in main and I would recommend not enabling the derivative script in your local instance.

@dziban303 - did you review these changes?

@g0Idfinger
Copy link
Contributor Author

I will make a branch today!

@Phaeton
Copy link
Collaborator

Phaeton commented Mar 22, 2024

Closing, superseded by #492

@Phaeton Phaeton closed this Mar 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
images Plane images planes Plane info
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants