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

Add initial documentation of focus feature system #3029

Merged
merged 4 commits into from
May 11, 2016
Merged

Conversation

kymckay
Copy link
Member

@kymckay kymckay commented Dec 13, 2015

As was discussed in the development meeting of 12/12/2015, we're implementing a new system of setting a specific goal to work towards between versions (dubbed 'project goals'). This is not a commitment of any kind, but rather a means of providing direction for collaborative development to focus in.

Initial explanation aims to make the intentions of the system clear and eliminate any potential for false expectations.

As was discussed in the development meeting of 12/12/2015, we're implementing a new system of setting a specific goal to work towards between versions (dubbed 'project goals'). This is not a commitment of any kind, but rather a means of providing direction for collaborative development to focus in.

Initial explanation aims to make the intentions of the system clear and eliminate any potential for false expectations.
@kymckay kymckay added this to the Ongoing milestone Dec 13, 2015
@kymckay
Copy link
Member Author

kymckay commented Dec 13, 2015

@acemod If you could check over this and make sure it matches what we discussed (make amendments as necessary) that'd be great 👍

Whenever a project goal is chosen:

* A new issue will be made for the purpose of discussing and documenting the current goal's development in a centralized location.
* A new branch originating from master will be made to develop collaboratively off of until the work is ready to be merged back in.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you elaborate on why that is required?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since releases can still occur in parallel to the development of these goals we can't have unfinished work sitting in the master branch

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This kind of goes against the mentality of creating changes that work independently. I feel like it would work a lot better to just continue with the previous way of doing things, but implement a way of directing progress towards the project goals.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see it as a slightly altered git flow workflow.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We will basically be continuing with the previous way of doing things

Having a branch off of master is exactly what we've been doing for features already 😄

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So basically there is no difference in the development process, just documentation of how it's been done so far anyways?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That fact that you're asking makes me think the documentation isn't clear enough 😝

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nope, it's just this line that confuses me 😁 :

Following the release of ACE 3.4.1

@thojkooi
Copy link
Contributor

Thanks @SilentSpike. I will look at this more closely in a few hours.

@commy2
Copy link
Contributor

commy2 commented Dec 15, 2015

👍 from me.

@kymckay kymckay changed the title Add initial documentation of project goal system Add initial documentation of focus feature system Jan 11, 2016

## 3 Branching and releases

We have a release scheduled every 2 weeks on a Tuesday. On the Friday before release, the project management will decide whether or not this scheduled release will continue. When continuing with the release, the current `master branch` will be merged into the `release branch`. The release branch will not contain any direct commits and no other branches will be merged into this branch. The exception being `hotfixes`, which are branched off `Release` and merged back into `Release` and `Master`.

`Hotfixes` are fixes for critical bugs that prevent stable gameplay with the currently available version of ACE3.

During this release process between the Friday and Tuesday, the day of release, work can continue on as normal on the `Master branch`. This includes new features, bug fixes that won't make it for release or other work. These will not be merged into the `Release branch` until the next release cycle, normally 2 weeks later.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we do it like that rather:

master branch
release branch

- Standardized formatting of branch names
- Removed outdated 2 week schedule information
- Removed unused branching strategy for project goals
# Conflicts:
#	wiki/development/branching-and-release.md
@kymckay
Copy link
Member Author

kymckay commented May 11, 2016

Think this is ready to be merged!

@bux
Copy link
Member

bux commented May 11, 2016

#3029 (comment)

... :)

@kymckay kymckay merged commit 4513c4f into gh-pages May 11, 2016
@kymckay kymckay deleted the goalsDoc branch May 11, 2016 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants