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

Rename "Agent" to "Compound" #67

Closed
bkloster opened this issue Nov 5, 2013 · 15 comments
Closed

Rename "Agent" to "Compound" #67

bkloster opened this issue Nov 5, 2013 · 15 comments

Comments

@bkloster
Copy link
Contributor

bkloster commented Nov 5, 2013

As noted in this forum post, agents are a special case of compounds. To avoid confusion and miscommunication between programmers and designers, all current occurences of "agent" should be replaced by "compound".

This includes:

  • Renaming the components and systems defined in src/microbe_stage/agent.*
  • Renaming the files src/microbe_stage/agent.* to src/microbe_stage/compound.*`
  • Searching all scripts inside res/scripts/microbe_stage for occurences of "agent" and replacing it with "compound"
@ChizuLaCheese
Copy link

I think I can do that if it's okay, so at the same time I try see what the looks like.

@ghost ghost assigned ChizuLaCheese Nov 19, 2013
@bkloster
Copy link
Contributor Author

I have assigned you. Be sure to have a look at the styleguide.

@ChizuLaCheese
Copy link

Ok, thanks.

@ChizuLaCheese
Copy link

And ther's no Ogre standard class called agent-something used right?

@bkloster
Copy link
Contributor Author

Nope. Last I checked, all occurrences of "[aA]gent" are related to our agent compound concept.

@ChizuLaCheese
Copy link

I've done it now, do I just click the green "New pull request" button?

@jjonj
Copy link
Contributor

jjonj commented Nov 22, 2013

Yep! From master to your branch, and we'll make sure everything is in order! btw. did you merge with master at any point? otherwise you'll want to do so and fix any new occurences of 'agent'.
I won't be able to look at it tonight tho.

@ChizuLaCheese
Copy link

Ok. I don't think I merged with master at any point. Ain't pull requests for merging?

@bkloster
Copy link
Contributor Author

Merges can go in two directions. Pull requests are for getting your changes into master. What jjonj is talking about is the other direction - getting any changes in master since you branched back into your branch. The latter should usually only be done right before accepting the pull request.

@ChizuLaCheese
Copy link

But won't that overide the changes that I made?

@bkloster
Copy link
Contributor Author

That's why it's called "merge", not "overwrite". Both changes are present afterwards. There will sometimes be conflicts where git can't figure out how to merge two divergent branches on its own, but it will tell the user about it and let them handle it.

@ChizuLaCheese
Copy link

Ok

@ChizuLaCheese
Copy link

Uhm, How do I make the merge? (https://github.com/ThrivingCheese/Thrive/pull/1)

@jjonj
Copy link
Contributor

jjonj commented Dec 18, 2013

You seem to have mixed two things up:
First thing is creating the pull request, this is done here on the Revolutionary-Games github, if you do it here you are requesting The official Thrives master branch to accept your feature branch into itsellf, instead of the other way around!

The second thing is merging master into your branch so your branch is up to date with the latest changes! This is not actually done with a pull request but by doing a "pull" with your local git installation, and pulling from the Revolutionary-Games master Thrive branch! Git will then tell you where it couldnt automatically merge the code together and you get "conflicts" which you will have to resolve by editing the files where it inserts something like:

<<<<<< HEAD
Your code version
=============
Other code version
>>>>>> Other branch

You needed to do both of these things (second one first) as well as "rebasing" which squishes all your commits into a single one so it looks cleaner when it actually gets merged into master.

It's all a rather complicated process, but it's what it takes to keep a project with a lot of people working on it, smooth.

Now regarding all the actual work here, since this issue appeared to die out, karthas077 did the renaming as he was already changing large parts of the code in his branch, so you wont need to do the process for this branch if everything goes smooth over there.

If you have any questions etc just ask, I'd be happy to help!

@ChizuLaCheese
Copy link

ok, thanks.

@jjonj jjonj self-assigned this Feb 21, 2014
@jjonj jjonj closed this as completed Apr 16, 2014
hhyyrylainen added a commit that referenced this issue May 4, 2021
and made it display as a PRE

closes #64
closes #67
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants