{"comment":{"url":"https://api.github.com/repos/rmp/libtext-sass-perl/comments/14423780","html_url":"https://github.com/rmp/libtext-sass-perl/commit/fa5c401215f475516d9a711afb3ba54e7adba9c2#commitcomment-14423780","id":14423780,"user":{"login":"rmp","id":19325,"avatar_url":"https://avatars.githubusercontent.com/u/19325?v=3","gravatar_id":"","url":"https://api.github.com/users/rmp","html_url":"https://github.com/rmp","followers_url":"https://api.github.com/users/rmp/followers","following_url":"https://api.github.com/users/rmp/following{/other_user}","gists_url":"https://api.github.com/users/rmp/gists{/gist_id}","starred_url":"https://api.github.com/users/rmp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rmp/subscriptions","organizations_url":"https://api.github.com/users/rmp/orgs","repos_url":"https://api.github.com/users/rmp/repos","events_url":"https://api.github.com/users/rmp/events{/privacy}","received_events_url":"https://api.github.com/users/rmp/received_events","type":"User","site_admin":false},"position":17,"line":652,"path":"lib/Text/Sass.pm","commit_id":"fa5c401215f475516d9a711afb3ba54e7adba9c2","created_at":"2015-11-16T19:10:22Z","updated_at":"2015-11-16T19:10:22Z","body":"\n> On 16 Nov 2015, at 09:32, Konstantin Tokarev <notifications@github.com> wrote:\n> \n> I wished to shorten this large $Text::Sass::Token:: prefix somehow, so used hack with local package. However, it would be needed to access tokens in Expr eventually...\n> \n> —\n> Reply to this email directly or view it on GitHub <https://github.com/rmp/libtext-sass-perl/commit/fa5c401215f475516d9a711afb3ba54e7adba9c2#commitcomment-14409498>.\n> \nMm, I appreciate the length but I switched it for a couple of reasons - perl-critic doesn’t like multiple packages in the same file and I don’t think CPAN::Kwalitee likes top-level modules like that, especially if they’re declared in a distribution bundle with a different name.\n\nMaybe just just using @EXPORT from Text::Sass::Token is simpler, or another import() override to allow runtime configuration.\n\n"}}