Skip to content

Rename LineModel.connections()->LineModel.edges()#119

Open
AsnelChristian wants to merge 1 commit intoMakman2:masterfrom
AsnelChristian:master
Open

Rename LineModel.connections()->LineModel.edges()#119
AsnelChristian wants to merge 1 commit intoMakman2:masterfrom
AsnelChristian:master

Conversation

@AsnelChristian
Copy link
Copy Markdown
Collaborator

First patch

@Makman2
Copy link
Copy Markdown
Owner

Makman2 commented Jun 9, 2017

MInd to close the issue inside the commit via a Closes xxxx reference?
Like coala I also like tags in commits ;) I think

LineModel: Rename connections() to edges()

as shortlog would fit pretty well.

I'm not so strict with shortlogs having only 50 chars max btw, I allow 72 like in the commit body. Usually you can write way better shortlogs if you have more space than 50 chars, especially when using tags ;)

@AsnelChristian
Copy link
Copy Markdown
Collaborator Author

done :)

@AsnelChristian AsnelChristian force-pushed the master branch 2 times, most recently from 0d60e99 to e4d1643 Compare June 9, 2017 17:36
@Makman2
Copy link
Copy Markdown
Owner

Makman2 commented Jul 29, 2017

Can still see some occurrences of connections :)

 mak  ~  dev  CE3D2   asnel/edge-rename  git grep -n connections
CE3D2/models/LineModel.h:13:    /// connections between single vectors.
CE3D2/models/LineModel.h:24:        /// vectors without connections.
CE3D2/models/LineModel.h:62:        /// define the line-connections of the model between the vectors.
CE3D2/models/LineModel.h:70:        /// define the line-connections of the model between the vectors.

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.

2 participants