Use S.? instead of S.?? all around Lift #1282

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@fmpwizard
Member

After I rebase #1233 Unify resource lookup using S.?? and S:.? into master, we are going to get a lot of warnings where we use S.??

This issue is to change all those areas to use S.? instead.

@fmpwizard fmpwizard was assigned Jun 22, 2012
@fbettag
Member
fbettag commented Jun 22, 2012

I guess a simple Regex would do:

find project/ -name '*.scala' | xargs sed -i.orig -e "s;S.??(;S.?(;g"

or something like that :)

Franz

Black Belt DevOp, Lift Committer and CEO.
Twitter: http://twitter.com/fbettag
GitHub: http://github.com/fbettag
Personal: http://www.uberblo.gs
Business: http://www.bett.ag

On Jun 22, 2012, at 5:26 AM, Diego Medina wrote:

After I rebase #1233 Unify resource lookup using S.?? and S:.? into master, we are going to get a lot of warnings where we use S.??

This issue is to change all those areas to use S.? instead.


Reply to this email directly or view it on GitHub:
#1282

@fmpwizard
Member

Yep, I ended up using IntelliJ's Find/Replace in Path, that took care of it, running tests now to make sure all is well.
Oh, I guess you can call S.?? but you cannot call S.? in unit tests, well, now to look at that and probably bring it up on the lift mailing list.

@fmpwizard fmpwizard closed this in bf4b646 Jul 3, 2012
@fmpwizard fmpwizard reopened this Jul 3, 2012
@fmpwizard fmpwizard closed this in a771f8b Jul 4, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment