Skip to content

[Fixes #636][5.1.x] Remove call to command rebuild_index#637

Merged
etj merged 1 commit into
5.1.xfrom
636_51x_rebuild
Jun 17, 2026
Merged

[Fixes #636][5.1.x] Remove call to command rebuild_index#637
etj merged 1 commit into
5.1.xfrom
636_51x_rebuild

Conversation

@etj

@etj etj commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@etj etj self-assigned this Jun 17, 2026
Copilot AI review requested due to automatic review settings June 17, 2026 15:26
@etj etj requested a review from mattiagiupponi June 17, 2026 15:26
@etj etj changed the title [Fixes #636][51x] Remove call to command rebuild_index [Fixes #636][5.1.x] Remove call to command rebuild_index Jun 17, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Removes the automatic invocation of Django’s rebuild_index management command from the migrations invoke task, aligning the task with environments where that command is unavailable and avoiding extra work/noisy failures during migrations.

Changes:

  • Removed the python manage.py rebuild_index call (and its surrounding try/except) from the migrations task.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@etj etj merged commit b9aa200 into 5.1.x Jun 17, 2026
3 checks passed
@etj etj deleted the 636_51x_rebuild branch June 17, 2026 15:40
etj added a commit that referenced this pull request Jun 17, 2026
(cherry picked from commit b9aa200)

Co-authored-by: Emanuele Tajariol <etj@geo-solutions.it>
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.

3 participants