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 automata.word to automata.Word #19210

Closed
dkrenn opened this issue Sep 14, 2015 · 7 comments
Closed

rename automata.word to automata.Word #19210

dkrenn opened this issue Sep 14, 2015 · 7 comments

Comments

@dkrenn
Copy link
Contributor

dkrenn commented Sep 14, 2015

word to Word to make it consistent

Depends on #19207
Depends on #19201
Depends on #18966
Depends on #19204
Depends on #19205
Depends on #19199
Depends on #19206
Depends on #18123

CC: @cheuberg @sagetrac-skropf

Component: finite state machines

Author: Daniel Krenn

Branch/Commit: u/cheuberg/fsm/upper-case-generator-word @ cab6fcc

Reviewer: Clemens Heuberger

Issue created by migration from https://trac.sagemath.org/ticket/19210

@dkrenn
Copy link
Contributor Author

dkrenn commented Sep 14, 2015

@cheuberg
Copy link
Contributor

@dkrenn
Copy link
Contributor Author

dkrenn commented Sep 14, 2015

Last 10 new commits:

7c9a164rephrase/extend doc of process_iterator_class
b580917Merge remote-tracking branch 'origin/u/dkrenn/fsm/languages' into fsm/words
da369d0#18123 comment 10, bullet 1: sentence deleted.
c02c375#18123 comment 10, bullet 2: delete unnecessary check
2301c1aTrac #18123 comment 10, bullet 3, 4, 6, 7: remove automatic_output_type=None
64ee3e8Trac #18123 comment 10, bullet 5: add additional doctests
c2dfc63Trac #19207: Merge #18123 to resolve merge conflict
5708d4aminor improvment of readability of example
3c0cc51rename: automata.word --> automata.Word
cab6fccTrac #19210: replace three more occurrences word->Word

@dkrenn
Copy link
Contributor Author

dkrenn commented Sep 14, 2015

Commit: cab6fcc

@dkrenn
Copy link
Contributor Author

dkrenn commented Sep 14, 2015

Dependencies: #19207, #19201, #18966, #19204, #19205, #19199, #19206, #18123

@cheuberg
Copy link
Contributor

comment:4

looks good to me. Checked for all occurrences of automata.word, AutomatonGenerators.word and `word`, everything else should have been revealed by doctests.

No deprecation as the code was merged in the last beta only.

@cheuberg
Copy link
Contributor

Reviewer: Clemens Heuberger

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