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

Remove old lookup code and obliterate the tables that it depended on #2765

Merged
merged 3 commits into from
Aug 26, 2021

Conversation

jcollins-g
Copy link
Contributor

This removes the old lookup code, the options used previously to fall back to it, and all the giant global tables it relied on.

@google-cla google-cla bot added the cla: yes Google CLA check succeeded. label Aug 26, 2021
@srawlins
Copy link
Member

minus ~850 lines of code 😁

Copy link
Member

@srawlins srawlins left a comment

Choose a reason for hiding this comment

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

Wow, so cool

import 'package:dartdoc/src/matching_link_result.dart';
import 'package:dartdoc/src/model/comment_referable.dart';
import 'package:dartdoc/src/model/model.dart';
import 'package:dartdoc/src/warnings.dart';
import 'package:markdown/markdown.dart' as md;
import 'package:meta/meta.dart';

const validHtmlTags = [
const _validHtmlTags = [
Copy link
Member

Choose a reason for hiding this comment

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

❤️

@@ -272,646 +228,9 @@ MatchingLinkResult _getMatchingLinkElementCommentReferable(
return MatchingLinkResult(lookupResult);
}

/// Returns null if element is a parameter.
Copy link
Member

Choose a reason for hiding this comment

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

👋 👋 👋

@jcollins-g jcollins-g merged commit b22cd23 into dart-lang:master Aug 26, 2021
@jcollins-g jcollins-g deleted the remove-old-lookup-code branch August 26, 2021 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Google CLA check succeeded.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants