Skip to content

repo sync#12065

Merged
Octomerger merged 4 commits intomainfrom
repo-sync
Nov 16, 2021
Merged

repo sync#12065
Octomerger merged 4 commits intomainfrom
repo-sync

Conversation

@Octomerger
Copy link
Copy Markdown
Contributor

This is an automated pull request to sync changes between the public and private repos.

🤖 This pull request should be merged (not squashed) to preserve continuity across repos, so please let a bot do the merging!

Peter Bengtsson and others added 4 commits November 16, 2021 20:22
* search improvements

* expose search result score on debug=1 qs (#22452)

* Search index improvements (#22428)

* concatenate headings for search ranking (#22445)

* search index improvements

* update index

* add data-search property used to create search records

* direct match title and normalize results to one per article

* the search index is getting huge

* add chinese lang, intro, and boost

* create a article level record with no heading for each article

* remove chinese language and exact match, boost breadcrumbs

* break out debug mode into separate branch

* break adding chinese out into another branch

* one record per article remove limits

* revert added spaces

* revert adding property on article page

* concatenate headings for search ranking

* remove slug property, fix test

* update record format in tests

* revert adding this property back

* scrape article-body not contents

* adding heading property to tests

* add headings to content property

* Update script/search/parse-page-sections-into-records.js

Co-authored-by: Peter Bengtsson <peterbe@github.com>

* Update script/search/parse-page-sections-into-records.js

Co-authored-by: Peter Bengtsson <peterbe@github.com>

Co-authored-by: Peter Bengtsson <peterbe@github.com>

* search by popularity (#22496)

* search by popularity

* adjust factors based on length of query and normalize Lunr score

* Apply suggestions from code review

Co-authored-by: Rachael Sewell <rachmari@github.com>

Co-authored-by: Rachael Sewell <rachmari@github.com>

* turn html into text with appropriate spaces (#22528)

* turn HTML into text with appropriate spaces

Part of #1141

* fix unit test

* improve tests

* small refactor

* slice the search results .content (#22514)

* search by popularity

* adjust factors based on length of query and normalize Lunr score

* slice the search result .content

Part of #1142

* Update lib/search/lunr-search.js

Co-authored-by: Rachael Sewell <rachmari@github.com>

Co-authored-by: Rachael Sewell <rachmari@github.com>

* truncate translated search index records (#22475)

* truncate translated search index records

* page entire page object

* revert search indexes

* try reverting search indexes again...

* out of sync with main in parent branch so reverting last commit again

* rever deleted indexes

* remove debug lines

* correct package-lock.json

* slice content string better (#22586)

Part of #1149

* include home pages in the search (#22568)

* Search remove hyphen as token (#22584)

* slice better when not present in content (#22672)

* fix missing search breadcrumbs (#22681)

* fix missing search breadcrumbs

* hopefully fix tests

Co-authored-by: Peter Bengtsson <mail@peterbe.com>

* Highlight based on combined matchData (#22692)

* highlight based on combined matchData

* refactoring

* remove results when lang or version changes (#22697)

* search with AND on title first and favor that more

* commented out code

* Update lib/search/lunr-search.js

* only apply the wildcard to the title search if the last word is short

Co-authored-by: Rachael Sewell <rachmari@github.com>
@Octomerger Octomerger requested a review from a team as a code owner November 16, 2021 20:48
@Octomerger Octomerger added automated-reposync-pr Auto label for repo sync autoupdate labels Nov 16, 2021
@github-actions github-actions Bot locked as spam and limited conversation to collaborators Nov 16, 2021
@Octomerger Octomerger enabled auto-merge November 16, 2021 20:48
@github-actions github-actions Bot added the triage Do not begin working on this issue until triaged by the team label Nov 16, 2021
@Octomerger Octomerger merged commit fad5576 into main Nov 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

automated-reposync-pr Auto label for repo sync triage Do not begin working on this issue until triaged by the team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant