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 scheduling to periodic #174

Merged
merged 4 commits into from
Jul 5, 2021

Conversation

fonsecadeline
Copy link
Contributor

No description provided.

@senhalil
Copy link
Contributor

senhalil commented Apr 8, 2021

@fonsecadeline can you merge all the commits of this PR so that renaming is done in one single commit?

@fonsecadeline fonsecadeline force-pushed the rename_scheduling_to_periodic branch 2 times, most recently from ecfb79c to e506ba7 Compare April 9, 2021 09:30
@senhalil senhalil added this to the v1.8 milestone Apr 15, 2021
@fonsecadeline fonsecadeline force-pushed the rename_scheduling_to_periodic branch 2 times, most recently from d6507dd to f585342 Compare April 20, 2021 12:54
lib/heuristics/periodic_heuristic.rb Outdated Show resolved Hide resolved
@fonsecadeline fonsecadeline force-pushed the rename_scheduling_to_periodic branch 2 times, most recently from 5406d64 to 4fe4411 Compare April 21, 2021 13:24
util/job_manager.rb Outdated Show resolved Hide resolved
@fonsecadeline
Copy link
Contributor Author

fonsecadeline commented Apr 27, 2021

@fonsecadeline fonsecadeline mentioned this pull request Apr 27, 2021
3 tasks
@fonsecadeline fonsecadeline force-pushed the rename_scheduling_to_periodic branch from 5ecc6f8 to 7dfcc1f Compare May 28, 2021 08:49
@fonsecadeline
Copy link
Contributor Author

@braktar @senhalil should I mention in CHANGELOG that we renamed scheduling heuristic to periodic heuristic ?

@senhalil
Copy link
Contributor

@braktar @senhalil should I mention in CHANGELOG that we renamed scheduling heuristic to periodic heuristic ?

I don't know, the heuristic was always called periodic on the user side, so there is no change from their point of view. From the dev point of view, it doesn't sound like it deserves to be on the changelog but either way is fine for me.

@fonsecadeline fonsecadeline force-pushed the rename_scheduling_to_periodic branch 3 times, most recently from 6f779a1 to 796afb6 Compare June 29, 2021 06:30
@senhalil senhalil merged commit 20b37c9 into Mapotempo:dev Jul 5, 2021
@fonsecadeline fonsecadeline deleted the rename_scheduling_to_periodic branch July 5, 2021 09:03
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.

3 participants